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

clearChanges method stopped working on 5.2.1 #5510

Closed
marciogurka opened this issue Nov 1, 2022 · 3 comments
Closed

clearChanges method stopped working on 5.2.1 #5510

marciogurka opened this issue Nov 1, 2022 · 3 comments
Assignees
Labels
bug Something isn't working forum Issues from forum high-priority Urgent to have fixed info requested Further information is requested regression Worked previously, now broken

Comments

@marciogurka
Copy link

Forum post

Check the example described by the user here

Add an item at the bottom using the text box. Check the BaseGridApp.Controls.Gantt.project.changes object
see that there is an item with a phantom id
press save in top left
now check BaseGridApp.Controls.Gantt.project.changes
the added task is still there but the Id has changed.
you can also try BaseGridApp.Controls.Gantt.project.clearChanges() which doesnt clear the tasks.

@marciogurka marciogurka added bug Something isn't working forum Issues from forum labels Nov 1, 2022
@marciogurka
Copy link
Author

User added a note

and a note the clearchanges works prior to sync to the server. So if you add an item and then clear changes it works. Soon as you sync it no longer goes away.

@isglass isglass added this to the 5.2.x milestone Nov 8, 2022
@matsbryntse matsbryntse added BLOCKING regression Worked previously, now broken labels Jan 4, 2023
@matsbryntse matsbryntse modified the milestones: 5.2.x, 5.2.7 Jan 11, 2023
@isglass isglass added the info requested Further information is requested label Jan 11, 2023
@isglass isglass modified the milestones: 5.2.7, 5.2.8 Jan 11, 2023
@SergeyMaltsev SergeyMaltsev modified the milestones: 5.2.8, 5.2.9 Jan 17, 2023
@arcady-zherdev
Copy link

Your test case at https://oneplanqa.azurewebsites.net/home/repro504 no longer works.
Could you please update it so I could investigate the issue? ..Since I can't reproduce it on our examples.

@matsbryntse matsbryntse added high-priority Urgent to have fixed and removed BLOCKING labels Jan 27, 2023
@matsbryntse matsbryntse removed this from the 5.2.9 milestone Jan 27, 2023
@arcady-zherdev
Copy link

This issue seems caused by id field mapping issue described in this ticket: #412
So closing this as duplicating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working forum Issues from forum high-priority Urgent to have fixed info requested Further information is requested regression Worked previously, now broken
Projects
None yet
Development

No branches or pull requests

5 participants