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

Finish renaming Display settings -> Preferences #28336

Closed
wants to merge 18 commits into from

Conversation

Aryan-81
Copy link
Collaborator

@Aryan-81 Aryan-81 commented Dec 23, 2023

Fixes: #26874
PR Finish renaming
display-settings-form -> preferences-form
zulip\web\styles\settings.css
zulip\web\templates\settings\display_settings.hbs

display-settings-radio-choice-label -> preferences-radio-choice-labeld
zulip\web\styles\settings.css
zulip\web\templates\settings\display_settings.hbs

emoji-display-settings-status -> emoji-preferences-status
zulip\web\src\settings_display.js

emoji-display-settings -> emoji-preferences
web\templates\settings\display_settings.hbs

display_settings_legacy -> preferences_legacy
zulip\zerver\actions\user_settings.py
zulip\zerver\lib\events.py
zulip\zerver\models\users.py
zulip\zerver\tests\test_events.py
zulip\zerver\tests\test_event_system.py

display_settings_labels -> preferences_labels
zulip\web\src\settings.js
zulip\web\src\settings_config.ts

get_all_display_settings -> get_all_preferences
zulip\web\src\admin.js
zulip\web\src\settings.js
zulip\web\src\settings_config.ts

_check_update_display_settings -> _check_update_preferences
zulip\zerver\lib\event_schema.py

display_settings -> preferences
zulip\web\src\admin.js
zulip\web\src\settings.js
zulip\web\templates\settings\display_settings.hbs
zulip\web\templates\settings\organization_user_settings_defaults.hbs
zulip\web\templates\settings\user_display_settings.hbs
zulip\web\tests\i18n.test.js

'display_settings.hbs' -> 'preferences.hbs'

display_settings_section -> preferences_section
zulip\web\e2e-tests\settings.test.ts

@Aryan-81 Aryan-81 changed the title changes display-settings-form -> preferences-form Finish renaming Display settings -> Preferences refer-#26874 Dec 23, 2023
@Aryan-81 Aryan-81 changed the title Finish renaming Display settings -> Preferences refer-#26874 Finish renaming Display settings -> Preferences refer- #26874 Dec 23, 2023
@Aryan-81 Aryan-81 changed the title Finish renaming Display settings -> Preferences refer- #26874 Finish renaming Display settings -> Preferences Dec 23, 2023
@zulipbot zulipbot added size: M and removed size: S labels Dec 24, 2023
@zulipbot zulipbot added size: L and removed size: M labels Dec 24, 2023
@zulipbot
Copy link
Member

Heads up @Aryan-81, we just merged some commits that conflict with the changes you made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/main branch and resolve your pull request's merge conflicts accordingly.

@timabbott
Copy link
Sponsor Member

Closing, since we must have merged work towards this on another PR, given the merge conflicts. In any case, the PR is very messy with merge conflicts and I'd recommend reading our Git guide to learn how to not do that.

Check out our GitHub guide and commit guidelines for more details.

@timabbott timabbott closed this Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finish renaming Display settings -> Preferences
3 participants