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

Settings sync stores more than 20 resource revisions #8589

Closed
jeanp413 opened this issue Mar 4, 2022 · 0 comments · Fixed by #8644
Closed

Settings sync stores more than 20 resource revisions #8589

jeanp413 opened this issue Mar 4, 2022 · 0 comments · Fixed by #8644
Assignees
@mustard-mh
Labels
aspect: browser IDE feature: settings sync feature: settings sync team: IDE

Comments

@jeanp413
Copy link
Member

jeanp413 commented Mar 4, 2022 •
edited
Loading

Bug description

Settings sync stores more than 20 resource revisions when it should only be 20 rev max

Steps to reproduce

gp_setting_sync
@loujaybee loujaybee added the feature: settings sync feature: settings sync label Mar 4, 2022
@akosyakov akosyakov moved this to Scheduled in 🚀 IDE Team Mar 7, 2022
@mustard-mh mustard-mh self-assigned this Mar 7, 2022
@mustard-mh mustard-mh mentioned this issue Mar 7, 2022
Merged
@akosyakov akosyakov moved this from Scheduled to In Review in 🚀 IDE Team Mar 8, 2022
@jeanp413 jeanp413 moved this from In Review to Done in 🚀 IDE Team Mar 12, 2022
@jeanp413 jeanp413 moved this from Done to Awaiting Deployment in 🚀 IDE Team Mar 12, 2022
@mustard-mh mustard-mh moved this from Awaiting Deployment to Done in 🚀 IDE Team Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mustard-mh mustard-mh

Labels
aspect: browser IDE feature: settings sync feature: settings sync team: IDE
Projects
🚀 IDE Team
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[server] remove setting sync record with limit gitpod-io/gitpod
3 participants
@loujaybee @mustard-mh @jeanp413