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.

Sign up for GitHub

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

Jump to bottom

Do not patch scheduleEntries if they didn't change #7036

Open
BacLuc opened this issue Mar 16, 2025 · 1 comment
Open

Do not patch scheduleEntries if they didn't change #7036

BacLuc opened this issue Mar 16, 2025 · 1 comment
Labels
Needs prototype Please make a prototype (code, mockup, sketch, etc.) first type: Frontend

Comments

@BacLuc
Copy link
Contributor

BacLuc commented Mar 16, 2025

If you add a schedule entry here:
Image
you might overwrite changes of other users on the existing scheduleentry.
Image

Its an edge case, but when we have no other issues, we can fix this.

Maybe we want to solve this generally in the form component, that can determine which fields of which entities are dirty, and only patch them.
@manuelmeister manuelmeister added Meeting Discuss Am nächsten Core-Meeting besprechen Needs prototype Please make a prototype (code, mockup, sketch, etc.) first and removed Meeting Discuss Am nächsten Core-Meeting besprechen labels Mar 16, 2025
@manuelmeister
Copy link
Member

Core Meeting Discuss

We are not sure what we want here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Needs prototype Please make a prototype (code, mockup, sketch, etc.) first type: Frontend
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@BacLuc @manuelmeister