#5039 Refresh the ticket description in real-time when it changes
Closed: Fixed 3 years ago by pingou. Opened 3 years ago by kparal.

Fedora QA implemented a blocker voting system using Pagure tickets:
https://pagure.io/fedora-qa/blocker-review

Each time a person sends a comment containing a vote (e.g. BetaBlocker +1), our bot intercepts the message, computes the current votes, and updates the ticket summary. This happens very fast, in a matter of seconds after the user vote was submitted.

The current problem is that each person who casts a vote needs to refresh the page manually again, to make sure their vote was counted (it wouldn't be counted if they made a typo, for example). That's tedious to do every time. People would like to see the effect happen automatically.

We would like Pagure to automatically refresh the ticket description, it if gets changed, for everyone having that page currently open in their browser.

Please note that it seems that Pagure maybe already attempts something like this already. Sometimes, after sending a comment, the ticket description changes to a "mess of characters" - something that appears to be a single line of unformatted characters. (I couldn't reproduce it right now, but when I can, I'll attach a screenshot). My guess is that the ticket description gets updated and Pagure tries to reflect that, but it doesn't reformat that using Markdown, and instead displays it as raw text (with linefeeds munched?). But this happens only sometimes, in other cases I see no change in the description. So it might be some race, a different bug, I don't know.


This is supposed to already happen, and I've observed the "unformatted blob" issue you're talking about. I would say this is a bug, actually.

Metadata Update from @ngompa:
- Issue tagged with: UI, bug

3 years ago

Basically, we're sending to the EV the raw text while maybe we should send the html, or find a way to have the raw text converted to html afterward.

I'm guess doing the first may be easier

Metadata Update from @pingou:
- Issue assigned to pingou

3 years ago

Metadata Update from @pingou:
- Issue set to the milestone: 5.12

3 years ago

Commit 63e9f55 fixes this issue

@pingou Can you please tell me when I can test this? It doesn't seem to work at the moment.

@pingou Can you please tell me when I can test this? It doesn't seem to work at the moment.

It'll be in pagure 5.12, we've fixed the live-refresh in prod but not the
formatting issue that we've fixed in this ticket.

Login to comment on this ticket.

Metadata
Attachments 1
Attached 3 years ago View Comment
Related Pull Requests
  • #5044 Merged 3 years ago