Skip to content

[WIP] Fix data-links confusion during view navigation#242

Closed
Copilot wants to merge 1 commit intomasterfrom
copilot/fix-data-links-navigation-issue
Closed

[WIP] Fix data-links confusion during view navigation#242
Copilot wants to merge 1 commit intomasterfrom
copilot/fix-data-links-navigation-issue

Conversation

Copy link
Copy Markdown

Copilot AI commented Nov 20, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

I need to summarize changes in this PR. The reason for this change is that when you navigate between different views but in the same project, data-links get confused. They get confused because when we initiate the document service, we construct it with a dummy editor document that later is replaced by a proper one loaded from the backend. When you normally visit the site it works fine, as datalinks and all other machinery gets subscribed to the document changes after we replace it with a proper document. But when you navigate between pages, the subcriptino already happened, and then briefely we start doing reactions on dummy document.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Nov 20, 2025
Copilot AI requested a review from pjurczynski November 20, 2025 09:13
@pjurczynski pjurczynski deleted the copilot/fix-data-links-navigation-issue branch November 20, 2025 11:37
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.

2 participants