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

Validating the invoice in the Invoices tab, then clicking save reverts the change to the invoice #221

Open
Thomasjanssens001 opened this issue Mar 4, 2023 · 1 comment
Assignees
Labels
bug Prio 1 The most important tickets

Comments

@Thomasjanssens001
Copy link
Collaborator

Thomasjanssens001 commented Mar 4, 2023

Visual:

image

Steps To Reproduce:

  1. navigeer naar ongevalideerde factuur
  2. klik op de blauwe validatieknop (1)
  3. klik vervolgens op de knop bewaren (2)

Expected Result:

De factuur blijft gevalideerd na het klikken van de knop "Bewaren"

Actual result

De validatie van de factuur wordt gereset.

@Thomasjanssens001 Thomasjanssens001 added bug Prio 1 The most important tickets and removed Prio 1 The most important tickets labels Mar 4, 2023
@Laoujin
Copy link
Member

Laoujin commented Mar 5, 2023

Looking good!

Op projecten wordt er voor environment / severity / prio typische met badges of tags ofzo gewerkt.
Op die manier kunnen lijsten gemakkelijk gefilterd worden op "high prio"
--> Ik zou severity & prio hier puur als badged modeleren en niet nogmaals in de tekst van de bug zelf opnemen (DRY => vraag er naar tijdens onze volgende stv:)
--> Environment mag je hier weglaten, aangezien er voor deze testen maar 1 environment is
--> De link naar de factuur details die je gebruikt hebt is altijd handig om vb bij Context extra te tonen!
--> Ook het tijdstip dat het gebeurd is, daarmee kunnen devs naar de logs gaan kijken


Screenshot:
Je duidt nu dingen aan (goed!) --> ik zou daar nog extra aan toevoegen om de meerwaarde van de foto te vergroten zoals vb dit:

image


Steps To Reproduce:
klik op de blauwe validatieknop "FACTUUR NU VERIFIËREN"
(vette toegevoegd)
--> In dit geval is het niet echt nodig omdat er maar 1 validatieknop is, op een grotere pagina kunnen er meerdere mogelijkheden zijn, het is daarom nooit slecht om heel specifiek te zijn en de labels van knoppen etc exact over te nemen in bug reports (of: als je werkt met pijlen kan je dit daarin verwerken. Vb: Pijl met tekst "(1) factuur valideren" en '(2) factuur bewaren'


Context
Wanneer een factuur gevalideerd wordt en er op de knop "bewaren" wordt geklikt, revert dit de validatie. Wanneer de validatieknop gebruikt wordt en de pagina daarna gerefreshed wordt, blijft de validatie geldig, zelfs na het gerbuik van de knop "Bewaren"

--> Ik zou dit volledig durven weglaten. Het is goed om een 1 liner te hebben die de bug omschrijft (zoals jij gedaan hebt met de title van de bug!), daarna is het niet nodig om dit nog eens in proza te zetten. Je zou daar vb direct de Steps to reproduce kunnen zetten.


Pas je de andere bugs ook aan?

Key Take Aways:

  • Keep it short & sweet.
  • Steps to reproduce is what everyone wants to see FIRST when opening a bug report
  • DRY: Don't Repeat Yourself (Use badges only)
  • Link to page with the bug
  • Add time it happened for log lookups

Ook een belangrijke:
Als je een bug gevonden hebt en je hebt de steps to reproduce neergepend, volg dan je eigen instructies eens vanaf 0 en tot op de letter om te zien of je dan effectief de bug triggert.
In deze applicatie gaat het typisch altijd straight forward zijn omdat het een relatief simpele applicatie is. Op een echt project gaan bugs zich pas na een aantal pre-acties manifesteren (vb eerst op een andere pagina iets anders doen)

@Thomasjanssens001 Thomasjanssens001 added the Prio 1 The most important tickets label Mar 15, 2023
@Runekid Runekid self-assigned this Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Prio 1 The most important tickets
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants