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: Add Organization, Sponsor, User, and Event Pages with Optimized Data #31

Draft
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

Anmol-Baranwal
Copy link
Member

Below are the preview of the page and the routes

Organization Page

route: /organization

Light Mode Dark Mode
organization_light organization_dark

Sponsor Page

route: /organization/sponsor

Light Mode Dark Mode
sponsor_light sponsor_dark

Event Page

route: /organization/event

Light Mode Dark Mode
event_light event_dark

EventName Page

route: /organization/event/eventName

Light Mode Dark Mode
eventName_light eventName_dark

User Page

route: /organization/user

Light Mode Dark Mode
user_light user_dark

Special Note

  • I made some minor adjustments, particularly to the header (to incorporate the Guidlify branding).
    • These changes are minor and can be easily modified.
  • While the colors and text size may not be exact, they closely resemble the Figma design.
  • I still need to complete the tab and timeline in the eventName page.
  • The code structure is optimal and well-organized with data, as shown below:
Header Header Header
image image image
  • The route can be changed, and the inner header is still pending. I used usePathname, which wasn't rendering images on the page.
  • There might be 1-2 individual components whose styling slightly differs from Figma, which we can address now.

We still need to make these responsive :)

Copy link
Contributor

@mkubdev mkubdev left a comment

Choose a reason for hiding this comment

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

Hi @Anmol-Baranwal 👋🏻
The figma implementation look good to me ! Thank you !

@darkterminal
Copy link
Contributor

@Anmol-Baranwal What the punk is this bro!!! You did a 🔥 to this project and I think this will burn the entire world!

Sorry, I think I am too much... but yes, of course! 🤣

Copy link
Member

@vinzvinci vinzvinci left a comment

Choose a reason for hiding this comment

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

So far so good 💯 Anmol never fail us 💯

@Anmol-Baranwal
Copy link
Member Author

Anmol-Baranwal commented Jan 11, 2024

Thanks :)
I will ask you guys to review it once every page is responsive. I plan to push the code every day. If you want, you can work on it as well. I'm currently working on the organization page and the user page.

@darkterminal
Copy link
Contributor

@Anmol-Baranwal If not you don't mind, can you make a mocking API using https://miragejs.com/. So I would look at the suggested data structures of the UI that you create, so that you stay focused on the frontend.

@Anmol-Baranwal
Copy link
Member Author

@Anmol-Baranwal If not you don't mind, can you make a mocking API using https://miragejs.com/. So I would look at the suggested data structures of the UI that you create, so that you stay focused on the frontend.

Sure. I will do it.

@darkterminal
Copy link
Contributor

That's will be amazing! let's 🪨 -in out...

@darkterminal
Copy link
Contributor

@Anmol-Baranwal I've added miragejs before at PR #26

@Anmol-Baranwal
Copy link
Member Author

Responsive styling of /organization, /organization/user, /organization/sponsor is done :)

Yay! Two more to go 😁

@Anmol-Baranwal Anmol-Baranwal marked this pull request as draft January 12, 2024 12:20
@Anmol-Baranwal
Copy link
Member Author

@darkterminal
Great refactoring into separate files. The file structure looks better now :)

I will do the same from next time.

@darkterminal
Copy link
Contributor

@Anmol-Baranwal 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants