-
Notifications
You must be signed in to change notification settings - Fork 156
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
[full-ci] Set document title consistently for all apps in useAppDefaults composable #6870
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
6e2cc87
to
bae8532
Compare
Results for oCISFiles2 https://drone.owncloud.com/owncloud/web/25272/57/1 💥 The acceptance tests failed on retry. Please find the screenshots inside ...
webUIFilesActionMenu-fileFolderActionMenu_feature-L14.pngwebUIFilesActionMenu-fileFolderActionMenu_feature-L37.pngwebUIFilesActionMenu-versions_feature-L15.pngwebUIFilesActionMenu-versions_feature-L26.pngwebUIFilesActionMenu-versions_feature-L77.pngwebUIFilesDetails-fileDetails_feature-L14.pngwebUIFilesDetails-fileDetails_feature-L142.pngwebUIFilesDetails-fileDetails_feature-L146.pngwebUIFilesDetails-fileDetails_feature-L173.pngwebUIFilesDetails-fileDetails_feature-L182.pngwebUIFilesDetails-fileDetails_feature-L27.pngwebUIFilesList-fileList_feature-L15.pngwebUIFilesList-fileList_feature-L22.pngwebUIFilesList-fileList_feature-L28.pngwebUIFilesList-fileList_feature-L33.pngwebUIFilesList-fileList_feature-L42.pngwebUIFilesList-fileList_feature-L55.pngwebUIFilesList-sort_feature-L31.pngwebUIFilesList-sort_feature-L51.pngwebUIFilesList-sort_feature-L72.pngwebUIFilesSearch-search_feature-L113.pngwebUIFilesSearch-search_feature-L122.pngwebUIFilesSearch-search_feature-L139.pngwebUIFilesSearch-search_feature-L146.pngwebUIFilesSearch-search_feature-L185.pngwebUIFilesSearch-search_feature-L27.pngwebUIFilesSearch-search_feature-L38.pngwebUIFilesSearch-search_feature-L95.png |
01d1050
to
6e6fa35
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - but please rebase once more since the last time is quite a while ago, sorry :-(
... and move makeRequest into its own composable.
SonarCloud Quality Gate failed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
... and move
makeRequest
to its own composable.Description
This refactors document title setting and moves
makeRequest
into its own composable.Related Issue
Motivation and Context
How Has This Been Tested?
addAppProviderFile
seems to produce the same request as before the refactor, but my ocis doesn't like either.. so I cannot be 100%. Could anyone with a working wopi setup give this a spin?Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: