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

verified flag is also updated on edited invoice object #242

Conversation

Runekid
Copy link
Collaborator

@Runekid Runekid commented Apr 6, 2023

issue #221

This whole edit screen along with some other edit screens should probably be refactored when we decide on how we want to handle forms, like we discussed. It's not great that the invoice object is split into different objects and can be updated separately like in this case for the verified flag on both the initInvoice and invoice objects

edited invoice object
@Runekid Runekid requested a review from Laoujin April 6, 2023 12:04
@Laoujin
Copy link
Member

Laoujin commented Apr 9, 2023

Yup, it needs to reviewed. I'll merge this as a temp work-around.

Note that modifying objects like this is typically a no-no in React since it potentially messes up the rendering.

@Laoujin Laoujin merged commit f1e0e62 into itenium-be:master Apr 9, 2023
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.

2 participants