cms icon indicating copy to clipboard operation
cms copied to clipboard

feat: setup entire login and course purchase setup

Open devsargam opened this issue 1 year ago • 12 comments

Resolves #169

This pr currently adds:

  • [x] Ability to signup in house
  • [x] Hash Password
  • [x] In house sign in
  • [x] Ability to purchase a course in house
  • [x] Change the schema for storing purchases
  • [x] Integrate Razorpay

In future:

  • [x] ~Verify the user with some method~ Not required for v0

devsargam avatar Mar 10 '24 06:03 devsargam

hey @hkirat, how should I proceed with mail verification? I can implement it my way but what would suit you the most???

devsargam avatar Mar 10 '24 06:03 devsargam

No need to verify emails for v0

hkirat avatar Mar 10 '24 07:03 hkirat

hey @hkirat, Do we need to support login with appx as a fallback method?

devsargam avatar Mar 10 '24 12:03 devsargam

Also, just to be sure, content is still comming from appx right?

devsargam avatar Mar 10 '24 12:03 devsargam

No all content is in house currently as well appx gives us

  1. authentication (they store user information)
  2. Let us know which user has purchased which course - ref - https://github.com/code100x/cms/blob/main/src/utiles/appx.ts#L15

hkirat avatar Mar 10 '24 12:03 hkirat

oh, makes a lot sense. Thanks

devsargam avatar Mar 10 '24 13:03 devsargam

https://github.com/code100x/cms/assets/76874341/a27839ce-12b4-415c-a390-6154c35f09d1

devsargam avatar Mar 11 '24 18:03 devsargam

@hkirat ready for review. Could you check this out?

cc: @siinghd

devsargam avatar Mar 11 '24 18:03 devsargam

As this is v0 a lot of things to be improved. LMK your thoughts on this and I shall continue.

PR already getting a lot large and messy.

devsargam avatar Mar 11 '24 18:03 devsargam

@hkirat hoping it comes to your notifs

devsargam avatar Mar 11 '24 19:03 devsargam

Will review it asap

hkirat avatar Mar 11 '24 19:03 hkirat

Just clarifying a few things I have left a bunch of TODOs which can be their own issue. Simple enough to fix not totally necessary but can make the app a lot better and performant not totally sure thooo

devsargam avatar Mar 11 '24 19:03 devsargam

too many merge confs, gonna close for now

hkirat avatar Apr 21 '24 21:04 hkirat

checking

hkirat avatar Jun 06 '24 08:06 hkirat