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

fix NotesModalButton #279

Merged
merged 2 commits into from
Jul 3, 2024
Merged

fix NotesModalButton #279

merged 2 commits into from
Jul 3, 2024

Conversation

Ninarchive
Copy link
Collaborator

Fixes a bug with the NotesModalButton on the EditInvoice page. When a user saves a note and immediately refreshes the page, the notes modal does not show the saved note.

This partially fixes #275

@JochemVH1
Copy link
Collaborator

Effe voor de duidelijkheid, dit fixed dus inderdaad dat als je F5 doet de comments wel beschikbaar zijn, maar zover ik begrepen heb als je deze code wegneemt en je drukt F5 dan is de store nog leeg wat er voor zorgt dat er een lege string als state wordt meegegeven aan:

const [text, setText] = useState<string>(value || '');

en useState gaat dan bij elke rerender een lege waarde terug geven.

door het volgende te doen:

if(!text && value){
    setText(value);
  }

gaat op de moment dat de store dan wel opgevuld is je via setText er voor zorgen dat useState wel de correcte text terug geeft na een rerender waardoor het werkt bij refresh.

In essentie zouden we dus het renderen van deze component moeten uitstellen tot op het moment dat de store wel een invoice heeft. lijkt mij een betere fix.

@Laoujin Laoujin merged commit c06f760 into master Jul 3, 2024
1 check passed
@Laoujin Laoujin deleted the fix-comments branch July 31, 2024 15:09
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

Successfully merging this pull request may close these issues.

[BUG] Adding comment to invoice
3 participants