-
-
Notifications
You must be signed in to change notification settings - Fork 423
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
[ui] [refactor] Drop MemberProfile #4100
Comments
I'd like to give this a try |
Hey, @benfurber. Should I pass the ProfileType down from UserProfile as a prop? Could I just get it from the user object itself? ![]() |
I can't seem to run cypress locally anymore. I get a SupaBase error. |
@CrowsVeldt you are probably missing the supabase setup: |
@mariojsnunes Thank you! |
Component infos
Description
We have
MemberProfile
for users who are the profileType of 'member' andSpaceProfile
for other profileTypes. The design ofMemberProfile
can be dropped andSpaceProfile
updated to allow for members. Thus then there's one Profile template. (Basically it's just the header area that should be different and the member badge.)Page related
Will be used on profiles.
Mockup
Example(s)
If you have any examples of existing similar components, add the link here. It can be a link to a code snippet and/or to a page of a projet/website where we can interact with the similar component.
Build suggestion
Could create some annoying merge conflicts if done at the same time as #4098 - this should probably be done before or after.
The text was updated successfully, but these errors were encountered: