Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

@max-nextcloud max-nextcloud commented Jan 4, 2023

Introduce hasEditor as reactive data to dynamically reload the MainContainer.

$editor is not reactive.
Once the MainContainer was rendered it stayed - even when the connection expired. The child components of MainContainer would still have the old $editor value.

Fixes #3374.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and test for the changes will be added to main branch later
  • Documentation (README or documentation) is not required

@cypress
Copy link

cypress bot commented Jan 4, 2023



Test summary

99 0 0 0Flakiness 0


Run details

Project Text
Status Passed
Commit 594874e ℹ️
Started Jan 4, 2023 11:34 AM
Ended Jan 4, 2023 11:39 AM
Duration 04:49 💡
OS Linux Ubuntu - 22.04
Browser Electron 106

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@max-nextcloud
Copy link
Collaborator Author

/compile

max-nextcloud and others added 2 commits January 4, 2023 12:32
Introduce `hasEditor` as reactive data to dynamically reload the MainContainer.

$editor is not reactive.
Once the MainContainer was rendered it stayed - even when the connection expired.
The child components of MainContainer would still have the old $editor value.

Fixes #3374.

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@max-nextcloud max-nextcloud force-pushed the backport/3627/stable25 branch from 4c8e1b9 to dc3dc19 Compare January 4, 2023 11:32
@juliusknorr juliusknorr changed the title fix: reconnect - reload main container [stable25] fix: reconnect - reload main container Jan 4, 2023
@juliusknorr juliusknorr merged commit 30c010f into stable25 Jan 4, 2023
@delete-merged-branch delete-merged-branch bot deleted the backport/3627/stable25 branch January 4, 2023 12:37
@skjnldsv skjnldsv mentioned this pull request Jan 6, 2023
5 tasks
@blizzz blizzz mentioned this pull request Jan 11, 2023
6 tasks
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.

4 participants