fix: Properly handle viewer loading in collectives

[juliusknorr]

• Fix emitting the loaded state in all cases
• Rework check for source based read only view to explicitly limit on versions or files with source only (without a file id)

🏁 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 the changes are covered with tests
• Documentation (README or documentation) has been updated or is not required

[cypress]

1 flaky tests on run #11599 ↗︎

0

149

2

0

1

Details:

fix: Properly handle viewer loading in collectives
Project: Text	Commit:549efefc71
Status: Passed	Duration: 04:22 💡
Started: Aug 9, 2023 8:17 AM	Ended: Aug 9, 2023 8:21 AM

cypress/e2e/api/SyncServiceProvider.spec.js • 1 flaky test

ViewOutputVideo

Test		Artifacts
Sync service provider > recovers from a dropped message		OutputScreenshots

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[mejo-]

Thanks a lot for tackling this!

[mejo-]

/compile