Skip to content
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

Comments with history #306

Open
Laoujin opened this issue Dec 26, 2024 · 0 comments
Open

Comments with history #306

Laoujin opened this issue Dec 26, 2024 · 0 comments

Comments

@Laoujin
Copy link
Member

Laoujin commented Dec 26, 2024

In plaats van een simpel comment field (of misschien beter: extra bovenop de bestaande comment velden)
Meerdere commentaren
Per commentaar bijhouden: wie, wanneer, tekst
Iedereen kan zijn eigen commentaar deleten/editeren maar enkel CAN_MANAGE_TIMESHEET_COMMENTS (etc) rechten kan van iedereen editeren/deleten.
Descending sort, laatste comment bovenaan.

Kan toevoegen, editeren, deleten.

Doe eerst een voortel hoe de UI voor deze feature eruit zou zien?

  • Hoe ziet een gebruiker of er comments zijn? (nu dikwijls een comment icon met drie puntjes erin + een tooltip)
  • Notitie staat dikwijls onderaan de pagina, dit staat best ergens bovenaan, zodat het goed zichtbaar is, makkelijk terug te vinden zonder te moeten scrollen
  • wat als er veel comments zijn? Misschien text filtering toevoegen zodat er makkelijk in gezocht kan worden?
  • Maak een aantal wireframes (kan getekend zijn of in paint of iets deftiger)

Just make the comments part of the entities -- don't store in a separate collection.
Have an interface for it so that the comments are the same everywhere.

Op volgende niveaus zouden we dat nodig hebben:
TOP PRIO:

  • ProjectMonth Timesheet
  • ProjectMonth itself

LOWER PRIO:

  • Projecten
  • Facturen

LOWEST PRIO:

  • Klanten

Deze features zijn optioneel, pas aan beginnen als de feature er al is, en dan eerst bespreken!

Extra (optioneel): een comment heeft "badges" (=tags)
Een badge heeft een "name" & een foreground/background color
De badges worden getoond bij de comment

Extra (optioneel): een @user
Wanneer er een @user is moet het duidelijk zijn voor die user dat hij ge-at is

  • eenmaal gelezen met die indicatie weggaan
  • moet iets herbruikbaar zijn zodat we die @ ook op andere plaatsen kunnen gebruiken
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant