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/update profile #2829

Merged
merged 1 commit into from
Dec 16, 2022
Merged

Feat/update profile #2829

merged 1 commit into from
Dec 16, 2022

Conversation

kyranjamie
Copy link
Collaborator

@kyranjamie kyranjamie commented Nov 9, 2022

Try out this version of the Hiro Wallet - download extension builds.

Clone of Friedger's PR with Ci permissions to create build

Todo

  • Use prod version of connect

@kyranjamie
Copy link
Collaborator Author

@friedger I've pushed the changes but the prettier, typescript, dependency checker, and unit tests are failing. When working locally, you can check these with yarn typecheck && yarn lint

@vercel vercel bot temporarily deployed to Preview November 23, 2022 13:34 Inactive
@markmhendrickson
Copy link
Collaborator

Should this build work okay for QA despite the failed checks?

@kyranjamie kyranjamie force-pushed the feat/update-profile branch 2 times, most recently from 8626b6b to d9c917f Compare December 7, 2022 07:57
@vercel vercel bot temporarily deployed to Preview December 7, 2022 08:01 Inactive
@kyranjamie kyranjamie requested review from edu-stx and fbwoolf and removed request for edu-stx December 7, 2022 08:06
@markmhendrickson
Copy link
Collaborator

LGTM!

@markmhendrickson markmhendrickson linked an issue Dec 12, 2022 that may be closed by this pull request
@kyranjamie kyranjamie force-pushed the feat/update-profile branch 2 times, most recently from 98b69e6 to f97f7f0 Compare December 13, 2022 09:23
@vercel vercel bot temporarily deployed to Preview December 13, 2022 09:26 Inactive
@kyranjamie kyranjamie force-pushed the feat/update-profile branch 2 times, most recently from a641db4 to 73fdb99 Compare December 13, 2022 09:40
@vercel vercel bot temporarily deployed to Preview December 13, 2022 09:45 Inactive
feat: fix import

fix: use correct ui components

fix: use whenStxChainId, use UI components

chore: upgrade stacks.js v5

fix: path url trailing slash

fix: revert accidential changes

chore: restore test files for profile updates

fix: fix resolution of @stacks/profile to 6.0.0

fix: restore event handler in content-script, apply prettier

fix: remove waitForNavigation

fix: revert wrong rebase changes

feat: add support for update-profile request

feat: fix import

chore: upgrade stacks.js v5

fix: path url trailing slash

fix: revert accidential changes

chore: restore test files for profile updates

fix: restore event handler in content-script, apply prettier

fix: remove duplicate imports

chore: add @stacks/profile
@vercel vercel bot temporarily deployed to Preview December 16, 2022 12:42 Inactive
@kyranjamie kyranjamie merged commit 17e624d into dev Dec 16, 2022
@kyranjamie kyranjamie deleted the feat/update-profile branch December 16, 2022 15:02
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.

Support updating the public profile
5 participants