Skip to content

[SG-912] Modify the mobile app to retrieve the user's avatar color#2284

Merged
BrandonM-Bitwarden merged 2 commits intomasterfrom
changes/SG-912-signed
Jan 12, 2023
Merged

[SG-912] Modify the mobile app to retrieve the user's avatar color#2284
BrandonM-Bitwarden merged 2 commits intomasterfrom
changes/SG-912-signed

Conversation

@BrandonM-Bitwarden
Copy link

@BrandonM-Bitwarden BrandonM-Bitwarden commented Jan 9, 2023

Type of change

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

Objective

Signed branch of https://github.com/bitwarden/mobile/pull/2277/files

Code changes

~src/App/Controls/AccountViewCell/AccountViewCellViewModel.cs: Added new field to avatar builder
~src/App/Controls/AvatarImageSource.cs: Added new field
~src/App/Controls/AvatarImageSourcePool.cs: Added new cache key value
~src/App/Pages/BaseContentPage.cs: adjusted state getter to pull once and for avatar
~src/App/Pages/BaseContentPage.cs: added refresh for avatar
~src/Core/Abstractions/IStateService.cs: Added new methods
~src/Core/Models/Domain/Account.cs: Added new field
~src/Core/Models/Response/ProfileResponse.cs: Added new field
~src/Core/Models/View/AccountView.cs: Added new field
~src/Core/Services/StateService.cs: Addded getters from profile
~src/Core/Services/SyncService.cs: added save for profile on sync

Screenshots

image

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

…2277)

* work: baseline

* fix: dont use profile for store

* fiix: use userid in key

* fix: lookup on AccountView list create

* fix my own bad advice + tweaks

* Autosync the updated translations (#2279)

Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>

* fix my own bad advice + tweaks

* fiix: use userid in key

* [PS-1352] Fix ignore diacritics in search (#2044)

* Fix ignore diacritics in search

This change updates the search function to ignore diacritical marks in search results. Marks are stripped from both the search input and results.

* Removed logs, added null or whitespace validation and improved formatting

Co-authored-by: aj-rosado <109146700+aj-rosado@users.noreply.github.com>
Co-authored-by: Andre Rosado <arosado@bitwarden.com>

* [PS-2145] add rainsee browser series support (#2272)

* fix: lookup on AccountView list create

* Autosync the updated translations (#2279)

Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>

* fix my own bad advice + tweaks

Co-authored-by: mp-bw <59324545+mp-bw@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>
Co-authored-by: noncenz <33376717+noncenz@users.noreply.github.com>
Co-authored-by: aj-rosado <109146700+aj-rosado@users.noreply.github.com>
Co-authored-by: Andre Rosado <arosado@bitwarden.com>
Co-authored-by: Auj625197595 <32589249+Auj625197595@users.noreply.github.com>
@BrandonM-Bitwarden BrandonM-Bitwarden self-assigned this Jan 9, 2023
@BrandonM-Bitwarden BrandonM-Bitwarden changed the title [SG-912] Modify the mobile app to retrieve the user's avatar color (#… [SG-912] Modify the mobile app to retrieve the user's avatar color Jan 9, 2023
@BrandonM-Bitwarden BrandonM-Bitwarden merged commit 6102a0c into master Jan 12, 2023
@BrandonM-Bitwarden BrandonM-Bitwarden deleted the changes/SG-912-signed branch January 12, 2023 18:27
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.

2 participants