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

Investigate multiple ApplyEdit requests from IDE on startup #5637

Closed
hubertp opened this issue Feb 13, 2023 · 1 comment
Closed

Investigate multiple ApplyEdit requests from IDE on startup #5637

hubertp opened this issue Feb 13, 2023 · 1 comment
Assignees
@farmaazon
Labels
--low-performance p-medium Should be completed in the next few sprints x-chore Type: chore
Milestone
Beta Release

Comments

@hubertp
Copy link
Collaborator

hubertp commented Feb 13, 2023

On a startup for a simple project (take the 0.up to 100 example project) IDE makes a total of 4(!) ApplyEdit requests.
The first request includes a text edit for the whole file.
Successive requests differ by "visualization":{"name":{"content":{"content":"JSON"}},"project":"Builtin"} vs "visualization":null for individual IDE node entries in the metadata section.

We need to investigate if such requests are actually necessary, especially on startup.
@hubertp hubertp added p-high Should be completed in the next sprint -gui labels Feb 13, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Feb 13, 2023
@enso-bot enso-bot bot mentioned this issue Feb 14, 2023
Closed
@jdunkerley jdunkerley added this to the Beta Release milestone Feb 21, 2023
@farmaazon farmaazon moved this from 📤 Backlog to ❓New in Issues Board Jul 3, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Jul 3, 2023
@jdunkerley jdunkerley added p-medium Should be completed in the next few sprints and removed p-high Should be completed in the next sprint labels Aug 30, 2023
@farmaazon farmaazon removed this from Issues Board Feb 12, 2024
@farmaazon
Copy link
Contributor

Fixed in GUI2: there we send only one update on new projects to set idmap and metadata.

@farmaazon farmaazon closed this as not planned Won't fix, can't repro, duplicate, stale Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@farmaazon farmaazon

Labels
--low-performance p-medium Should be completed in the next few sprints x-chore Type: chore
Projects
None yet
Milestone
Beta Release
Development

No branches or pull requests
3 participants
@hubertp @farmaazon @jdunkerley