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

Rename setting name to reduce its length #32998

Merged
merged 1 commit into from
Jun 27, 2022
Merged

Rename setting name to reduce its length #32998

merged 1 commit into from
Jun 27, 2022

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Jun 23, 2022

Follow up of #31932

DB is limiting setting name to 64, but this was 72.

  • Remove legacy jQuery event listener as those settings are not part of the web UI anymore

@artonge artonge added bug 3. to review Waiting for reviews labels Jun 23, 2022
@artonge artonge added this to the Nextcloud 25 milestone Jun 23, 2022
@artonge artonge self-assigned this Jun 23, 2022
@artonge
Copy link
Contributor Author

artonge commented Jun 23, 2022

/backport to stable24

@artonge
Copy link
Contributor Author

artonge commented Jun 23, 2022

/backport to stable23

@CarlSchwan CarlSchwan added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 23, 2022
@blizzz
Copy link
Member

blizzz commented Jun 24, 2022

/drone/src/tests/lib/Collaboration/Collaborators/UserPluginTest.php:825
/drone/src/apps/settings/tests/Settings/Admin/SharingTest.php:150
/drone/src/apps/settings/tests/Settings/Admin/SharingTest.php:150

are failing

@artonge artonge force-pushed the fix/setting_name branch from d048f10 to 7e29001 Compare June 27, 2022 09:01
Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge force-pushed the fix/setting_name branch from 7e29001 to 347ca20 Compare June 27, 2022 09:04
@artonge
Copy link
Contributor Author

artonge commented Jun 27, 2022

I forgot some rename here and there, all green now.

I also removed some legacy frontend note that are linked to the setting but not needed anymore.

@artonge artonge merged commit 0fe7064 into master Jun 27, 2022
@artonge artonge deleted the fix/setting_name branch June 27, 2022 09:58
@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants