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

More profile details & Edit #625

Merged
merged 16 commits into from
Jan 4, 2025
Merged

More profile details & Edit #625

merged 16 commits into from
Jan 4, 2025

Conversation

kernelkind
Copy link
Member

Adds more profile details
image

and EditProfileView:
image

The profiles don't update in notedeck, but I can see the changes in Damus iOS when I make a change from notedeck. I think this is a nostrdb bug because when I do a profile update in Damus iOS, I also don't see it in notedeck.

Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
Signed-off-by: kernelkind <[email protected]>
@kernelkind
Copy link
Member Author

kernelkind commented Jan 4, 2025

CI is red but will pass when https://github.com/damus-io/nostrdb-rs/tree/client-events is merged. Now the profile updates can be seen on notedeck after making a modification

@kernelkind
Copy link
Member Author

Screen.Recording.2025-01-04.at.2.10.52.PM.mov

@kernelkind
Copy link
Member Author

closes #162

@jb55
Copy link
Contributor

jb55 commented Jan 4, 2025

can you post light mode screenshots?

@kernelkind
Copy link
Member Author

image image

@jb55 jb55 merged commit e1187c3 into damus-io:master Jan 4, 2025
3 of 7 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.

2 participants