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

[full-ci] Vue 2.7 #7877

Merged
merged 6 commits into from
Nov 15, 2022
Merged

[full-ci] Vue 2.7 #7877

merged 6 commits into from
Nov 15, 2022

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Oct 26, 2022

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
vue (source) 2.6.14 -> 2.7.14 age adoption passing confidence

Configuration

πŸ“… Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

β™» Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

πŸ”• Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@update-docs
Copy link

update-docs bot commented Oct 26, 2022

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.

@dschmidt
Copy link
Member

This at least needs vue-template-compiler updated at the same time.
It's probably also blocked by MartinMalinda/vue-concurrency#82

@renovate renovate bot force-pushed the renovate/vue-monorepo branch 2 times, most recently from 06a7791 to c32e0f6 Compare October 27, 2022 11:01
@dschmidt dschmidt force-pushed the renovate/vue-monorepo branch 2 times, most recently from dcff70b to 0191c11 Compare October 27, 2022 15:03
@dschmidt
Copy link
Member

@dschmidt
Copy link
Member

dschmidt commented Oct 27, 2022

I've updated the vue-template-compiler and removed @vue/composition-api from the dependencies and all usages of its default export (which is used to install it as a plugin).
To continue to work on this PR one can simply search '@vue/composition-api' and replace it in the whole project with 'vue'.
Additionally one should use the vue-2.7 branch of the owncloud-design-system.

After doing that the (unit) test suite looks fairly okay already:

Test Suites: 13 failed, 158 passed, 171 total

I expect this to go down quite a bit already after vue-concurrency is fixed. Apart from that mainly tests that were already broken with composition-api enabled before are still broken now and lack their workaround. Those could be fixed one by one to keep the scope of this PR more limited.

@dschmidt dschmidt force-pushed the renovate/vue-monorepo branch 2 times, most recently from 795783c to f81d795 Compare November 11, 2022 21:16
@dschmidt dschmidt changed the title fix(deps): update dependency vue to v2.7.13 fix(deps): update dependency vue to v2.7.14 Nov 12, 2022
@dschmidt dschmidt changed the title fix(deps): update dependency vue to v2.7.14 [full-ci] Vue 2.7 Nov 12, 2022
@ownclouders
Copy link
Contributor

ownclouders commented Nov 12, 2022

Results for oCISSharingPerm2 https://drone.owncloud.com/owncloud/web/29770/66/1

πŸ’₯ The acceptance tests failed on retry. Please find the screenshots inside ...

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L47.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L47.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L48.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L48.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L49.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L49.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L50.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L50.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L51.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L51.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L52.png

webUISharingFilePermissionMultipleUsers-shareFileWithMultipleUsers_feature-L52.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L54.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L54.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L55.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L55.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L56.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L56.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L57.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L57.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L58.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L58.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L59.png

webUISharingFilePermissionsGroups-sharePermissionsGroup_feature-L59.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L47.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L47.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L48.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L48.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L49.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L49.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L50.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L50.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L51.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L51.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L52.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L52.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L53.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L53.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L54.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L54.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L55.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L55.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L56.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L56.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L57.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L57.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L58.png

webUISharingFolderAdvancedPermissionMultipleUsers-sharedFolderWithMultipleUsersAdvancedPermissions_feature-L58.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L54.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L54.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L55.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L55.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L56.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L56.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L57.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L57.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L58.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L58.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L59.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L59.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L60.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L60.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L61.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L61.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L62.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L62.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L63.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L63.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L64.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L64.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L65.png

webUISharingFolderAdvancedPermissionsGroups-shareAdvancePermissionsGroup_feature-L65.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L48.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L48.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L49.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L49.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L50.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L50.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L51.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L51.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L52.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L52.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L53.png

webUISharingFolderPermissionMultipleUsers-shareFolderWithMultipleUsers_feature-L53.png

@dschmidt dschmidt marked this pull request as draft November 12, 2022 16:10
@dschmidt dschmidt force-pushed the renovate/vue-monorepo branch 4 times, most recently from cf7458c to d00b84e Compare November 14, 2022 17:01
Copy link
Collaborator

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice πŸš€ 2 small things I noticed:

@dschmidt dschmidt mentioned this pull request Nov 14, 2022
10 tasks
@kulmann kulmann marked this pull request as ready for review November 14, 2022 20:41
@sonarcloud
Copy link

sonarcloud bot commented Nov 14, 2022

SonarCloud Quality Gate failed.Β  Β  Quality Gate failed

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

61.4% 61.4% Coverage
0.2% 0.2% Duplication

import { queryItemAsString, useAppDefaults, useRouteQuery } from 'web-pkg/src/composables'
import { defineComponent } from '@vue/composition-api'
import { defineComponent } from 'vue'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be united with line 39, no? πŸ₯· Not worth an individual commit though, was just browsing and spotted this. Awesome work btw!

Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice ❀️

@@ -53,8 +48,12 @@ export default defineComponent({
const store = useStore()
const sharesLoading = computed(() => store.getters['Files/sharesTreeLoading'])

const { incomingParentShare, ...rest } = useIncomingParentShare()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not so happy with var name rest, but also don't have a better idea... πŸ€”

@dschmidt dschmidt merged commit 64a20a8 into master Nov 15, 2022
@delete-merged-branch delete-merged-branch bot deleted the renovate/vue-monorepo branch November 15, 2022 10:24
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.

None yet

5 participants