Charcoal-SE / metasmoke

Web dashboard for SmokeDetector.
https://metasmoke.erwaysoftware.com
Creative Commons Zero v1.0 Universal
43 stars 34 forks source link

Flag Logs empty #561

Closed RobCoding closed 5 years ago

RobCoding commented 5 years ago

On CHQ I mentioned to Makyen that the "Flag Logs" are empty, he confirms that this is a problem for him too.

Conversation from CHQ:

@Makyen When I go to "MS" and click on the 🏳️ it takes me to this webpage: https://metasmoke.erwaysoftware.com/post/153411/flag_logs for the Logs, but there are always no entries; even if I'm a flagger.

@Rob Yeah, that link has never worked for me either. I've assumed it's a bug, because the data is available from the API.

Link to conversation on CHQ: https://chat.stackexchange.com/transcript/message/48447938#48447938

We are only able to click on something when there is a 'flag icon' (🏳️) it would be great to be able to click on the title too.

The main concern of this Bug Report is that occasionally a flag icon appears, but clicking on it takes us to a 'template' webpage which has room for entries (presumably our own flags and any others we have permission to see) but the webpage isn't populated with the information that we were expecting to see.

This is what I see:

"Flag logs for In CA, Old car registration/smog is Due, Can I sell the car? User Post Result Eligible Flaggers

We are Charcoal ...".

Clicking on the "Eligible Flaggers" link simply dismisses it, and no listing is provided.

Expected operation: Clicking on the title or IF the rare flag icon appears clicking on that should provide some historical information.

When clicking on the flag icon and being presented with the "Eligible Flaggers" link clicking on it should do something instead of nothing.

ArtOfCode- commented 5 years ago

If you see a filled flag icon, the post had autoflags cast on it; clicking the icon will take you to that page, which will show you the autoflag records.

If you see an empty flag icon, the post had manual or "other" flags cast on it; clicking the icon will still take you to that page, but it only shows autoflags - you'll see no logs.

Half-bug, half-FR.

makyen commented 5 years ago

Sounds like this could be fixed in a few different ways:

If there are manual flags and no autoflags:

  1. The empty flag is not clickable.
  2. The empty flag is not displayed.
  3. The page linked by the empty flag shows the users that flagged. This information is available from the MS API, so is considered public.
  4. On the /flag_logs page change "Flag logs" to "Autoflag logs". Perhaps add a line of text explicitly saying that manual flags are not displayed.

If 3 is done, then it would be reasonable to display the manual flags in addition to the autoflags when both are present.

ArtOfCode- commented 5 years ago

I think I originally intended to do 3, then deployed it before I managed and promptly forgot... [status-on-the-todo-list]

makyen commented 5 years ago

Yeah, this is not a critical issue, but it does appear it's causing some confusion. When I was first looking at those pages I ended up comparing what was returned from the MS API for manual flags and what was displayed on that page. Doing so confirmed for me that the issue was just that the page didn't display manual flags, rather than that the record of manual flags had been lost/didn't exist.

IMO, this isn't a big issue, but clearly is causing some confusion. A stop-gap that would help reduce confusion would be to change what is displayed on the /flag_logs page from "Flag logs for" to "Autoflag logs for". That would make it clear that the listing is only intended to include autoflags and not manual flags. I've added that as option 4 in my list above.

RobCoding commented 5 years ago

Undo and myself had a discussion about this earlier: https://chat.stackexchange.com/transcript/message/48462065#48462065 - it's the "looky Lou's confusion" (UI ambiguity) that has recently cropped up.

While it's more involved option 3 and the comment about it ("If 3 is done, then it would be reasonable to display the manual flags in addition to the autoflags when both are present.") seems best - IF there are any flags from any source and they are not hidden from view by some means (such as Moderator/CM's actions on a particular site) then we should be easily (and non-confusingly) be able to view everything neatly organized together.


In regards to Art's first comment: "... If you see an empty flag icon, the post had manual or "other" flags cast on it; clicking the icon will still take you to that page, but it only shows autoflags - you'll see no logs." :

Sometimes when I have a flag that is manually provided (either by typing in Chat or from the prior page, by hitting the ✓ ✗ 💩 icons) I can see an entry in the logs (and it's a manual flag!).

Next to it there is a link that says "delete" and if I want to delete my flag I click on that - doing so takes me to a webpage that says that is unauthorized and to contact the administrator.

I can force the delete by simply going to Chat and replying to the SD Report using an 'opposite feedback'.

So fixing the delete link, (for the manual flags that appear where only auto flags are expected) is another confusing aspect to this so-called simple bug.

It's probably a lack of clarity and the inconsistency that is leading to the questions/problem. The pages need 'help tips' (❔) that mobile uses can tap on (that aren't broken, like the existing implementation) so we can get hints - or we could have a Doc Link in the footer.

A requirement to read all the source code and know years of history behind how things work isn't very user friendly, and can also lead to confusion for veteran users.

ArtOfCode- commented 5 years ago

Some misunderstanding there, @RobCoding. Feedback != flags. You can provide feedback from chat or with the ✓ ✗ 💩 buttons; you can also delete your own feedback. That's distinct from flags, which are a separate entity and aren't listed on the feedback page.