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: content wiping out in description in subactivity card is now fixed #843

Closed
wants to merge 1 commit into from

Conversation

srvEq
Copy link
Contributor

@srvEq srvEq commented Dec 6, 2024

No description provided.

@srvEq srvEq requested a review from nesadrian December 6, 2024 04:27
@srvEq srvEq linked an issue Dec 6, 2024 that may be closed by this pull request
@nesadrian
Copy link
Contributor

Screen.Recording.2024-12-06.at.14.40.25.mov

I'm still having this issue. I would recommend taking a look at the way the API is called. I can see that there is a patchDescription, patchDuration and patchRole that are all calling the same endpoint that updates all the fields simultaneously. I believe the reason we are having this issue is that we are calling this endpoints with outdated values.

From the top of my head I think we could either include all the fields when updating a node (this can be problematic if multiple people are updating different fields in a node at the same time). Another option is to change the backend so that it only updates the fields we send in the request. If the backend already does this then we need to change our requests to only include the fields we want to change, and not all of them.

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.

Content been wiped when enter in another field in SubActivity card
2 participants