Skip to content

[PM-3834] Fix settings dialogs verbiage#2758

Merged
fedemkr merged 3 commits intovault/pm-2658/feature-settings-reorganizationfrom
vault/pm-3834/fix-settings-dialogs-verbiage
Sep 18, 2023
Merged

[PM-3834] Fix settings dialogs verbiage#2758
fedemkr merged 3 commits intovault/pm-2658/feature-settings-reorganizationfrom
vault/pm-3834/fix-settings-dialogs-verbiage

Conversation

@fedemkr
Copy link
Member

@fedemkr fedemkr commented Sep 11, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Fix settings dialogs verbiage and to actually open them.

Code changes

  • General: Changed dialogs to new strings and added confirmation dialogs to several links from settings.

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@fedemkr fedemkr requested a review from a team September 11, 2023 17:13
…lt/pm-3834/fix-settings-dialogs-verbiage

# Conflicts:
#	src/App/Resources/AppResources.resx
@fedemkr fedemkr requested a review from mpbw2 September 18, 2023 14:48
@fedemkr fedemkr merged commit f37df8c into vault/pm-2658/feature-settings-reorganization Sep 18, 2023
@fedemkr fedemkr deleted the vault/pm-3834/fix-settings-dialogs-verbiage branch September 18, 2023 14:50
fedemkr added a commit that referenced this pull request Sep 27, 2023
* [PM-2658] Settings Reorganization Init (#2697)

* PM-2658 Started settings reorganization (settings main + vault + about)

* PM-2658 Added settings controls based on templates and implemented OtherSettingsPage

* PM-2658 Fix format

* [PM-3512] Settings Appearance (#2703)

* PM-3512 Implemented new Appearance Settings

* PM-3512 Fix format

* [PM-3510] Implement Account Security Settings view (#2714)

* PM-3510 Implemented Security settings view

* PM-3510 Fix format

* PM-3510 Added empty placeholder to pending login requests and also improved a11y on security settings view.

* PM-3511 Implemented autofill settings view (#2735)

* [PM-3695] Add Connect to Watch to Other settings (#2736)

* PM-3511 Implemented autofill settings view

* PM-3695 Add Connect to watch setting to other settings view

* [PM-3693] Clear old Settings approach (#2737)

* PM-3511 Implemented autofill settings view

* PM-3693 Remove old Settings approach

* PM-3845 Fix default dark theme description verbiage (#2759)

* PM-3839 Fix allow screen capture and submit crash logs to init their state when the page appears (#2760)

* PM-3834 Fix dialogs strings on settings (#2758)

* [PM-3834] Fix import items link (#2782)

* PM-3834 Fix import items link

* PM-3834 Fix import items link, removed old link.

* [PM-4092] Fix vault timeout policies on new Settings (#2796)

* PM-4092 Fix vault timeout policy on settings for disabling controls and reset timeout when surpassing maximum

* PM-4092 Removed testing hardcoding of policy data
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.

2 participants