Skip to content

fixes done - #935

Merged
Cedarich merged 4 commits into
StayLitCodes:mainfrom
Swayymalcolm99:main
Oct 18, 2026
Merged

Cedarich merged 4 commits into
StayLitCodes:mainfrom
Swayymalcolm99:main

Conversation

@Swayymalcolm99

Copy link
Copy Markdown
Contributor

Closes #819

Summary

  • Add editable display name and avatar upload to Settings, with a 2 MB image limit.
  • Persist profile details and per-event email/in-app notification preferences in browser storage.
  • Show and allow copying the connected public key.

Validation

  • Frontend lint: passed
  • Frontend production build and TypeScript validation: passed
  • Jest: not part of frontend CI; local run has unrelated existing failures, including Playwright tests picked up by Jest and an MSW ESM import issue.

Profile and notification preferences are currently browser-local; public key display reflects the single active wallet supported by the current context.

## Summary
- Add editable display name and avatar upload to Settings, with a 2 MB image limit.
- Persist profile details and per-event email/in-app notification preferences in browser storage.
- Show and allow copying the connected public key.

## Validation
- Frontend lint: passed
- Frontend production build and TypeScript validation: passed
- Jest: not part of frontend CI; local run has unrelated existing failures, including Playwright tests picked up by Jest and an MSW ESM import issue.

Profile and notification preferences are currently browser-local; public key display reflects the single active wallet supported by the current context.
@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@Swayymalcolm99 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Cedarich
Cedarich merged commit f8472b4 into StayLitCodes:main Oct 18, 2026
4 checks passed
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.

[Frontend] User profile & settings page (avatar, notification prefs, linked wallets) (150)

3 participants