cms icon indicating copy to clipboard operation
cms copied to clipboard
verified publisher icon code100x

added change password feature

Open AuenKr opened this issue 1 year ago • 5 comments

PR Fixes:

  • Added change password directly from app.100xdevs.com while maintaining password synchronization with harkirat.classx.co.in

Assumption env variable APPX_AUTH_KEY = appxapi APPX_CLIENT_SERVICE = Appx

UI /password change Password Page

Resolves

  • #748

Checklist before requesting a review

  • [x] I have performed a self-review of my code
  • [x] I assure there is no similar/duplicate pull request regarding same issue

AuenKr avatar Oct 05 '24 22:10 AuenKr

Hey, I have created the same pr But it got closed by maintainers the only difference is mine works on OTP not in current password. https://github.com/code100x/cms/pull/1415

atul24112001 avatar Oct 06 '24 03:10 atul24112001

@devsargam , Earlier I also created PR for the same feature #749 . That was not matching the current UI, so created a new one that does not have conflicts.

AuenKr avatar Oct 06 '24 16:10 AuenKr

@hkirat thoughts on this? else closing this pr within a week

devsargam avatar Oct 08 '24 21:10 devsargam

@devsargam @hkirat, I just want to know why it was closed previously.

previous reason:

  1. The password is not in sync with appx.
  2. Appx does not expose any endpoint for that.

as both reasons are resolved, is there any other reason for not changing the password directly from cms instead of using the appx site?

AuenKr avatar Oct 12 '24 07:10 AuenKr

This probably is not available from server side if I am not wrong but I will get to kirat regarding it

devsargam avatar Oct 12 '24 07:10 devsargam