-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
adds a completed user profile section and fixes newsletter styles #68
base: main
Are you sure you want to change the base?
adds a completed user profile section and fixes newsletter styles #68
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks ok thank you, i will attach more commits with this PR directly ok
No problem, you can play around with the colors, I'm not sold on the light theme for it but its up to you |
OK, thank you very much to be get involved this issue :) @jakebogan01 |
@jakebogan01 Hey how are you ? I hope everything is going well recently with you. i did here with angular with these component by using ionic framework in the base This is a single user-itemSCSS This is a list of users |
@Honzoraptor31415 What do you think about this PR ? |
yeah, I'm looking at the commits now, |
Basically, if you do similar design here with the app then i will implement API to show realtime data in our homepage. It should be responsive as well. This PR comes from the issue: #67 |
@xuelink I added a user completed section, I commented it out on the home page for now. Inside of the component inside the loop, I added some comments to better understand what is going on.