Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Displayed value should be that with highest verification number #882

Closed
edwardchalstrey1 opened this issue Jul 26, 2022 · 1 comment
Closed

Comments

@edwardchalstrey1
Copy link
Contributor

... or the most commonly entered value, rather than the most recent

@tomalrussell
Copy link
Contributor

This is not at all obvious to me - why should the displayed value not be the most recently edited?

What implications does this have for editing flows, from a user perspective:

  • What happens when I edit an already-verified value? Does that new value show in any way?
  • How do I see if a non-verified or less-verified value has been set?
  • If one value is verified, how can I assert that another value should be the correct one?
  • If an edit is malicious (however mildly so) and verified, how would I correct or remove it?
  • How is "most commonly entered" calculated?
  • How can I understand what values have been entered, how commonly, and how often verified?

And then in implementation on the backend:

  • currently, all edits are recorded in the edit history, and the current value for each attribute is stored in the buildings table
  • currently, all verifications (per user, building, attribute) are stored in another table; verification count for the current is calculated in the application code
  • what data model changes would be required?
  • what API changes would be required?

@edwardchalstrey1 edwardchalstrey1 moved this from To do: Ordered priorities to Development Backlog: Everything that we want to add to the platform but is not a priority in 2022 Turing Research Engineering - Colouring London Development Jul 26, 2022
@colouring-cities colouring-cities locked and limited conversation to collaborators Jul 26, 2022
@edwardchalstrey1 edwardchalstrey1 converted this issue into discussion #883 Jul 26, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
2022 Turing Research Engineering - Co...
Development Backlog: Everything that ...
Development

No branches or pull requests

2 participants