open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
381 stars 202 forks source link

Bug: Avatar link broken when logged out user logs in on StarSearch page #3523

Closed nickytonline closed 4 weeks ago

nickytonline commented 1 month ago

If you're logged in, your avatar for StarSearch always appears.

CleanShot 2024-06-05 at 07 37 06

However, if you start from a logged out experience and log in, your avatar is broken.

CleanShot 2024-06-05 at 07 42 25

This wasn't an issue before, but now that we let users access /star-search and prompt them to log in if not logged in it's broken because we're retrieving the userId via getServerSideProps which only renders those props on the initial page load. The userId prop should be promoted to a variable within the component to handle the logged out then logged in scenario.

github-actions[bot] commented 1 month ago

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

open-sauced[bot] commented 4 weeks ago

:tada: This issue has been resolved in version 2.33.1-beta.1 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

open-sauced[bot] commented 4 weeks ago

:tada: This issue has been resolved in version 2.34.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: