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

feat: update for item login #1407

Merged
merged 10 commits into from
Aug 26, 2024
Merged

feat: update for item login #1407

merged 10 commits into from
Aug 26, 2024

Conversation

pyphilia
Copy link
Contributor

@pyphilia pyphilia commented Aug 8, 2024

closes #388

@pyphilia pyphilia requested review from spaenleh and removed request for spaenleh August 8, 2024 16:04
@pyphilia
Copy link
Contributor Author

pyphilia commented Aug 9, 2024

@spaenleh I updated all graasp packages for the chatbox, but it looks like this now, not sure if it's intended, so I've left it as is:
Screenshot 2024-08-09 at 14 23 23

@spaenleh
Copy link
Member

@pyphilia I fixed this in the chatbox repo. it was caused by the removal of the sx prop on the avatar component and so the margin was not applied anymore. I wrapped them in a stack with a gap instead.

@spaenleh spaenleh added the feature New feature or request label Aug 21, 2024
@spaenleh spaenleh marked this pull request as ready for review August 21, 2024 14:16
Copy link

sonarcloud bot commented Aug 22, 2024

@spaenleh spaenleh requested a review from ReidyT August 22, 2024 09:49
Copy link
Contributor

@ReidyT ReidyT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@spaenleh spaenleh added this pull request to the merge queue Aug 26, 2024
Merged via the queue into main with commit 3e7f6db Aug 26, 2024
5 checks passed
@spaenleh spaenleh deleted the update-for-item-login branch September 11, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request v2.37.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Email sort does not work in Membership table
3 participants