Update supervisor's idea table immediately upon scheduling a first meeting #105
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "first-meeting-feedback"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Before, after scheduling a first meeting, they had to refresh the entire page to show the information in the table.
Fixes #82
How to test
eve@example.com
Deployed to https://scipro-first-meeting-feedback.branch.dsv.su.se
@ -173,0 +172,4 @@
FirstMeeting saved = firstMeetingRepository.save(getModelObject());
// After saving the first meeting we have to populate it on the already loaded idea to
// make sure that other places that want to render the first meeting get the correct data.
// An alternative would be to detach the idea model to force a database refresh.
Can the first meeting be edited from other users or from other places?
If it can maybe we should enforce a database refresh as per your comment of an alternative method.
If not then this is good in that we don't need to do a database refresh on the view we are in.
Yes it can be edited from other places (but always by the supervisor). However, I do not see how that is relevant? The issue is the table not updating when using the modal dialog to schedule a meeting.
Do you want the page with the table to update itself based on what happens on a different page if it happened to be open simultaneously?
We discussed yesterday (2025-02-11) in person, and I am happy with the resolution we got to. So we keep it as is.
👍
PO review OK