openrightsgroup / blocked-org-uk

Template front-end code, markup, style-sheets, images and other assets for the Censorship Monitoring Project (blocked.org.uk)
https://www.blocked.org.uk/
GNU General Public License v3.0
13 stars 5 forks source link

Remove injunction flag information #333

Closed JimKillock closed 6 years ago

JimKillock commented 6 years ago

Create ability to remove the information from a flag (record created in error), or create a new flag (new status).

Normally a flag should persist as a prior observation, rather than be removed entirely

JimKillock commented 6 years ago

Mistakes, for ref:

http://couchtuner.mx http://couchtuner.video http://itswatchseries.to

need unflagging / deleting the flag entry

JimKillock commented 6 years ago

Could we also add a "checked" flag with date, so that we can record sites as observed " block appears correct"?

I think this would make the data more robust.

dantheta commented 6 years ago

Added. There is now an "unflag" button on the flag screen. Changes to the flag options are also written to a history table, which is shown below the editor form.

JimKillock commented 6 years ago

Thanks Dan, can we have the ability to delete a particular history entry (because the observation was incorrect) or mark the observation as incorrect?

JimKillock commented 6 years ago

Unfortunately there are quite a few entries I will need to delete, so this will be quite important to get the data up to scratch.

dantheta commented 6 years ago

No problem, I'll have that fixed tonight.

On 6 April 2018 15:17:13 BST, JimKillock notifications@github.com wrote:

Unfortunately there are quite a few entries I will need to delete, so this will be quite important to get the data up to scratch.

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

JimKillock commented 6 years ago

Thanks, if you can add an alert symbol where there is any prior flagging that would also be handy (on the index page showing all the sites per injunction). That would let me scan down the list to see what needs to be double checked, if an old in-error flag has been recorded and removed. Else, I will have to check each entry.

dantheta commented 6 years ago

Deleting a historical flag is now possible (there is a trashcan link in the history section).

The number of flags for a URL is now shown on the URL listing screen as well.

JimKillock commented 6 years ago

Thanks Dan. One final thing we could do:

This would help us to do re-checking in the future.

dantheta commented 6 years ago

Already added :) There's a "block appears correct" option in the list of flag options.

JimKillock commented 6 years ago

That's great Dan: although as it is not a error, it needs to be omitted from the list and calculations at

https://www.blocked.org.uk/legal-blocks/errors

(eg, not appear in "errors by type" and in the total of "sites incorrectly blocked")

Also, they should not appear in the list of incorrect blocks and not in the per-ISP totals.

I was thinking this might be a bit complicated … would it be easier to do as a different flag? Up to you of course.

JimKillock commented 6 years ago

Alternatively: you could leave it as is, but:

That seems simpler and you may have already done the first two?

dantheta commented 6 years ago

Ah yes. Those are done now.