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

29042021 move tags into lists #5043

Merged
merged 3 commits into from
May 17, 2021
Merged

Conversation

pascalwengerter
Copy link
Contributor

Description

See changelog

@pascalwengerter
Copy link
Contributor Author

Can someone review this with OC10 + resharing activated? My setup is broken regarding this currently 🥴

@kulmann
Copy link
Member

kulmann commented May 5, 2021

Can you rebase this PR?

@pascalwengerter pascalwengerter force-pushed the 29042021_move-tags-into-lists branch from e3d34c2 to d02eb85 Compare May 5, 2021 21:14
@pascalwengerter pascalwengerter requested a review from kulmann May 5, 2021 21:15
@ownclouders
Copy link
Contributor

💥 Acceptance tests TrashbinDelete failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15306/

20210505-212242-147.png
20210505-212314-940.png
20210505-212346-691.png
20210505-212417-880.png
20210505-212448-985.png
20210505-212519-982.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests TrashbinFilesFolders failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15306/

20210505-212241-335.png
20210505-212312-830.png
20210505-212344-167.png
20210505-212414-757.png
20210505-212445-843.png
20210505-212516-372.png
20210505-212546-204.png
20210505-212616-639.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests MarkdownEditor failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15306/

20210505-212502-924.png
20210505-212534-675.png
20210505-212606-529.png
20210505-212638-451.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests SharingPublicDifferentRoles failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15306/

20210505-212348-732.png
20210505-212415-497.png
20210505-212443-645.png
20210505-212511-804.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests webUISharingInternalUsersRoot2 failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/web/15306/

20210505-212533-382.png
20210505-212622-168.png
20210505-212856-809.png
20210505-212925-151.png

fschade and others added 3 commits May 17, 2021 10:37
add vue-axe to see a11y violations in the console while developing
add eslint rule from vue-a11y project to get linting errors on accessibility violations
@pascalwengerter pascalwengerter force-pushed the 29042021_move-tags-into-lists branch from 3360c64 to 2d1f051 Compare May 17, 2021 09:37
@pascalwengerter
Copy link
Contributor Author

Rebased again and ready for re-review @kulmann

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.

LGTM 👍

@kulmann kulmann merged commit 7bc2b1d into a11y-swarming May 17, 2021
@delete-merged-branch delete-merged-branch bot deleted the 29042021_move-tags-into-lists branch May 17, 2021 11:41
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