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

De-duplicate event handling to prevent errors on Draw-io #8576

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

diocas
Copy link
Contributor

@diocas diocas commented Mar 9, 2023

If users navigated out of Draw-io and returned afterwards, old event handlers were kept, which failed trying to look for iframes that no longer existed.
This fix removes the handlers when exiting, preventing these user visible errors.

@ownclouders
Copy link
Contributor

ownclouders commented Mar 9, 2023

Results for acceptance oCIS https://drone.owncloud.com/owncloud/web/33573/55/1
💥 The oCISSharingBasic tests pipeline failed. The build has been cancelled.

@ownclouders
Copy link
Contributor

ownclouders commented Mar 9, 2023

Results for acceptance oC10 https://drone.owncloud.com/owncloud/web/33573/73/1
💥 The oC10IntegrationNotifications tests pipeline failed. The build has been cancelled.

@kulmann
Copy link
Member

kulmann commented Mar 13, 2023

@diocas could you rebase your PR please? Notifications acceptance tests have been removed in master.

@diocas diocas force-pushed the up_drawio_load_bug branch from 4d1e8f8 to 8d84102 Compare March 13, 2023 14:05
@sonarcloud
Copy link

sonarcloud bot commented Mar 13, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@kulmann kulmann merged commit e66d876 into owncloud:master Mar 13, 2023
@micbar micbar mentioned this pull request May 3, 2023
89 tasks
@tbsbdr tbsbdr mentioned this pull request Jun 30, 2023
37 tasks
@tbsbdr tbsbdr added this to the CERN Web Merge milestone Jul 10, 2023
@diocas diocas deleted the up_drawio_load_bug branch July 24, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants