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

profile: Display email #291

Closed
sirpengi opened this issue Aug 31, 2023 · 3 comments · Fixed by #844
Closed

profile: Display email #291

sirpengi opened this issue Aug 31, 2023 · 3 comments · Fixed by #844
Assignees

Comments

@sirpengi
Copy link
Contributor

sirpengi commented Aug 31, 2023

In the profile screen (#195 ) we need to show the user's email address if it is accessible, but not the fake email that Zulip creates for the api.

Also need to handle the user update event for null values of delivery_email appropriately.

Helpful links:

@MrunalrajRedij

This comment was marked as off-topic.

@gnprice

This comment was marked as off-topic.

@gnprice
Copy link
Member

gnprice commented Jul 2, 2024

Also need to handle the user update event for null values of delivery_email appropriately.

This is accomplished by #784. So once that's merged, User.deliveryEmail can be used normally.

@sm-sayedi sm-sayedi self-assigned this Jul 22, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jul 27, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Aug 5, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Aug 5, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Aug 6, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Aug 8, 2024
@gnprice gnprice closed this as completed in ba952c3 Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants