Skip to content

implemented-googleOauth-and-todo11-crud-implemented-for-various-modules #14

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

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

DhruvKaria07
Copy link

Hello @anshumandas, This is Dhruv from DJSCE!
My PR consists of todo's 3,4 and 11: GoogleAuth and crud operations for various modules respectively.

TODO 3-4, expected integrating flutter-fire and implementing Google-Auth with which I've also tried to enhance the login_view. Here are some snips for your reference:

image
image

TODO 11, expected implementing CRUD operations on the server-side for role_requests, initializing firebase-datastore, Nested Category, Sub-Category tree creation with which additionally I've also added controllers, bindings and dummy products for cart module as well and I can assure that with minute changes and integration on the client-side the operations implemented would be good to go. Every module has been ensured proper routing, auth steps and has been mentioned in app_pages, app_routes as well.
Some images for your reference:

image
image

Please review the above PR, any feedback or comments would be greatly appreciated.
Thankyou for your time and consideration!

Hardik and others added 23 commits July 11, 2024 03:05
Refactor and Organize Asset Constants by HardikSJain
I had already added firebase.json to .gitignore, but I'm not sure how it got committed. 
Have removed it now.
…rd-email-verification

Feature/reset password email verification by Nemin Sheth
fix: bottom navigation current index by HardikSJain
@anshumandas
Copy link
Owner

pl take the latest from main trunk and resolve the conflicts for me to accept the PR

@DhruvKaria07
Copy link
Author

Hi @anshumandas, I've resolved all the conflicts and the branch looks ready to be merged.
Thankyou for your time!

@DhruvKaria07
Copy link
Author

DhruvKaria07 commented Jul 20, 2024 via email

@anshuman
Copy link

anshuman commented Jul 22, 2024

Hey @anshuman, I've resolved the conflicts and the branch looks ready to be merged. @here #14

On Sat, Jul 20, 2024 at 12:14 PM Anshuman Das @.> wrote: pl take the latest from main trunk and resolve the conflicts for me to accept the PR — Reply to this email directly, view it on GitHub <#14 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4U3T5XNMJHZ2M2RS2KGBJ3ZNIBNPAVCNFSM6AAAAABLEBXL3CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBQHE3DENJYGE . You are receiving this because you authored the thread.Message ID: @.>

@ITDhruv You've tagged a wrong person. Pls see to it.

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.

5 participants