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

Integrated Firebase Authentication #68

Merged
merged 5 commits into from
Oct 6, 2024

Conversation

Nkovaturient
Copy link
Contributor

@Nkovaturient Nkovaturient commented Oct 5, 2024

Issue : #3

event1

event5

1)Implemented the following authentication methods:-

  • Signup
    event2

  • Login
    event3

  • GoogleAuth

2)Alongside, enhanced code modularity and assets as well as styling for better retrieval of data as and when required

3)Sticky Header and enhanced Footer styling + improved UI/UX for seamless user experience

event4

Copy link

vercel bot commented Oct 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eventmint ❌ Failed (Inspect) Oct 6, 2024 5:40pm

@samyakmaitre
Copy link
Owner

I need a preview of it first, then we'll move forward.

@Nkovaturient
Copy link
Contributor Author

I need a preview of it first, then we'll move forward.

Sure, I will present you with a demo video.

@Nkovaturient
Copy link
Contributor Author

I need a preview of it first, then we'll move forward.

Hello @samyakmaitre ,

Here is a demo video on seamlessly integrated Auth Setup via firebase— Signup, Login, GoogleAuth methods alongwith enhanced UI/UX integration:-

EventMint.mp4

@Nkovaturient
Copy link
Contributor Author

Also, I would like to follow your advice on backend setup and search functionality, data configuration implementation.
Can you guide me on how to achieve it (like extra feature, or as such) as an apt setup for our eventmint web app?!

@samyakmaitre
Copy link
Owner

As of now only keep the signup/login authentication , remove the black theme change. I will merge it.

@Nkovaturient
Copy link
Contributor Author

As of now only keep the signup/login authentication , remove the black theme change. I will merge it.

Hey there @samyakmaitre ,
Changed and Committed!, ✅

Screenshot (237)

@samyakmaitre
Copy link
Owner

samyakmaitre commented Oct 6, 2024

I dont want to change the prior appearance, kindly refer to the image for reference. other than this everything is perfect.
Ignore bookmyshow name.

image

@Nkovaturient
Copy link
Contributor Author

I dont want to change the prior appearance, kindly refer to the image for reference. other than this everything is perfect. Ignore bookmyshow name.

image

Roger that! I will get back to you with update soon! 🚀👨‍💻

@Nkovaturient
Copy link
Contributor Author

Nkovaturient commented Oct 6, 2024

Hey there @samyakmaitre !
I have accomplished as you desired including navbar, header and footer-theme aligned. 😄🌟

Screenshot (239)

Screenshot (240)

@samyakmaitre
Copy link
Owner

Kindly unstage footer changes, I just need your signup/login authentication coz footer is handeled by another dedicated contributor.
Thank you for your cooperation, your PR is under lvl3 DW. just make these changes.
I myself am eager to merge this 🤣

@Nkovaturient
Copy link
Contributor Author

Kindly unstage footer changes, I just need your signup/login authentication coz footer is handeled by another dedicated contributor. Thank you for your cooperation, your PR is under lvl3 DW. just make these changes. I myself am eager to merge this 🤣

ah! Sure.. 🙂🙂

@Nkovaturient
Copy link
Contributor Author

Kindly resolve this conflicts since I have changed footer in the same commit and havent updated my forked repo with your main repo changes. I am facing reference broken error hence I had to revert to the last commit as of now.
Holy! 🙂🙏

@samyakmaitre samyakmaitre merged commit fd4f40f into samyakmaitre:main Oct 6, 2024
2 of 3 checks passed
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.

2 participants