-
Notifications
You must be signed in to change notification settings - Fork 176
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] change: remove the settings ui #5463
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. |
a81d0b8
to
ef4eef7
Compare
I had a very close look and couldn't find any more occurrences of the settings ui in config, docs, CI and of course the code itself. Please check carefully in review. |
Guess it deserves a changelog item (but perhaps adding one was scheduled for later :) )? |
ef4eef7
to
a0cab68
Compare
Added one, thanks for the reminder :-) |
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.
Good stuff. Hard to check whether it's complete as I'm on my phone right now, but all changes look good on their own 👍️
a0cab68
to
bd9e6bc
Compare
ownCloud Web recently transitioned to Vue3. The settings ui is still written in Vue2. Since it's pretty much unused we won't take the efforts of upgrading it to Vue3.
bd9e6bc
to
8c033e1
Compare
Kudos, SonarCloud Quality Gate passed! |
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 so far. Let us move on.
Description
ownCloud Web recently transitioned to Vue3. The settings ui is still written in Vue2. Since it's pretty much unused we won't take the efforts of upgrading it to Vue3. The only setting ("Language") has been implemented in the
account
page of ownCloud Web (reachable via user menu in the top right -> click on the username).Types of changes
Checklist: