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

feat: Save profile identity in one json #13937

Conversation

MishkaRogachev
Copy link
Contributor

Close #13767

What does the PR do

  • Store identity data in json (dedicated from profile showcase data)
  • Remove some deprecated code

Affected areas

Profile

Screenshot of functionality (including design for comparison)

Screen.Recording.2024-03-12.at.14.02.55.mov

@MishkaRogachev MishkaRogachev linked an issue Mar 12, 2024 that may be closed by this pull request
@status-im-auto
Copy link
Member

status-im-auto commented Mar 12, 2024

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 37addf9 #1 2024-03-12 13:13:14 ~6 min tests/nim 📄log
✔️ 37addf9 #1 2024-03-12 13:13:24 ~6 min macos/aarch64 🍎dmg
✔️ 37addf9 #1 2024-03-12 13:18:20 ~11 min tests/ui 📄log
✔️ 37addf9 #1 2024-03-12 13:19:06 ~12 min macos/x86_64 🍎dmg
✔️ 37addf9 #1 2024-03-12 13:23:56 ~17 min linux/x86_64 📦tgz
✔️ 37addf9 #1 2024-03-12 13:33:19 ~26 min windows/x86_64 💿exe

@MishkaRogachev
Copy link
Contributor Author

I was originally thinking of using one json for all profile data, but since the UI has separate flags for profile showcase and identity, I think it's better to split the save into two parts

Copy link
Member

@caybro caybro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@noeliaSD noeliaSD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM! I'm fine with 2 different json files one for identity and one for the rest of preferences.

I've tested it locally and it works fine!

@MishkaRogachev MishkaRogachev merged commit 611d70a into master Mar 13, 2024
8 checks passed
@MishkaRogachev MishkaRogachev deleted the feat/issue-13767-save-all-profile-preferences-in-one-json branch March 13, 2024 09:46
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.

Save all profile preferences in one JSon
4 participants