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

updated avatar component name, added ability to change username (only works server side for now), updated signin page, updated Footer #91

Merged
merged 19 commits into from
Dec 28, 2022

Conversation

rsandb
Copy link
Contributor

@rsandb rsandb commented Dec 27, 2022

A bunch of these commits were already added so I'm not sure why they're showing up here. The only one that I'm requesting is "de86903"

Removed quotation marks from the content. Were previously from being commented out.
Wrapped the content in p tags and removed the apostraphe
@fozziethebeat fozziethebeat added website UI/UX All website Components labels Dec 28, 2022
website/src/pages/account/index.tsx Outdated Show resolved Hide resolved
website/src/pages/account/edit.tsx Outdated Show resolved Hide resolved
website/src/pages/account/edit.tsx Show resolved Hide resolved
website/src/pages/account/edit.tsx Outdated Show resolved Hide resolved
@fozziethebeat
Copy link
Collaborator

Will approve after I see the conflicts are merged in and the small style changes are made.

Copy link
Collaborator

@AbdBarho AbdBarho left a comment

Choose a reason for hiding this comment

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

Thanks!

website/src/middleware.ts Outdated Show resolved Hide resolved
website/src/pages/404.tsx Outdated Show resolved Hide resolved
website/src/pages/_app.tsx Outdated Show resolved Hide resolved
website/src/pages/account/edit.tsx Outdated Show resolved Hide resolved
website/src/pages/account/index.tsx Outdated Show resolved Hide resolved
website/src/pages/api/username.tsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@AbdBarho AbdBarho left a comment

Choose a reason for hiding this comment

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

Thanks!

LGTM

We will have to wait for @fozziethebeat to approve.

Copy link
Collaborator

@k-nearest-neighbor k-nearest-neighbor left a comment

Choose a reason for hiding this comment

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

Looks great!

@fozziethebeat fozziethebeat merged commit f9ce7f9 into LAION-AI:main Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX All website Components website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants