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

Minimized calls to /getAddressProfile for auth user addresses #9128

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

mzparacha
Copy link
Contributor

@mzparacha mzparacha commented Sep 5, 2024

Link to Issue

Closes: #9053

Description of Changes

We were making a request to /getAddressProfile for each address per community of the auth user. This PR minimizes those API calls and utilizes the auth user profile/addresses data that was already fetched on mount.

"How We Fixed It"

N/A

Test Plan

  • Ensure auth user profile data (specially username, profile link and avatar) displayed anywhere has no regressions.
  • Ensure there are minimal calls to /getAddressProfile after app load.

Deployment Plan

N/A

Other Considerations

N/A

@mzparacha mzparacha self-assigned this Sep 5, 2024
@github-actions github-actions bot marked this pull request as draft September 5, 2024 16:23
@mzparacha mzparacha marked this pull request as ready for review September 5, 2024 16:24
@jnaviask jnaviask requested a review from kurtisassad September 5, 2024 18:21
@mzparacha mzparacha merged commit 541bf46 into master Sep 6, 2024
9 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.

Duplicated getAddressProfile calls
3 participants