-
Notifications
You must be signed in to change notification settings - Fork 447
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
Verifying Old password before updating new password.. #1930
Comments
@sk9331657 Can you be more specific? |
@kushthedude On the password update page , before updating password old password must be verified. |
Can i work on it or is anybody working on it ? |
Yeah sure 👍 |
@vaibhavmatta See if you can help here. |
@kushthedude Sure Sir..! I am working on it now.! |
@kushthedude Can you please specify the files responsible for this issue? |
I suggest you read more about emberjs before working on bugs or feature |
I've read about it about an hour ago..! Just give me some idea to start and I will resolve it. I am not getting about which Js file is responsible for password management and which front-end file (html or hb) is responsible for settings page. |
Thats why I told you, read about ember templates and components . |
Sir @kushthedude I've read everything about ember templates and components and it really helped me to learn about the directory system of our project. Please confirm, Should I add only the Old password input space as the frontend or should I work on its backend as well? Like if the old password is correct only then the new password will be submitted. |
@vaibhavmatta first of all dont call me sir, secondly just add a field in rhe FE which would check for old password. |
@kushthedude Is anyone working on this? |
Nope
…On Fri, 27 Sep, 2019, 17:09 Aaditya Joshi, ***@***.***> wrote:
@kushthedude <https://github.com/kushthedude> Is anyone working on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1930?email_source=notifications&email_token=AKQMTLQ6X2P2IN6LLMVU2EDQLXWGBA5CNFSM4GP5VOTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7YT6YA#issuecomment-535904096>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKQMTLR756KR6YXHNEQ242TQLXWGBANCNFSM4GP5VOTA>
.
|
@kushthedude Hi! I would love to work on this issue. Could you please assign the issue to me. |
@zbeproud Thankyou, You can start working on this issue. Feel free to ask any problems you face. |
Sure. I will get right on top of it! |
I will be able to proceed with solving this as soon as I can log in to the local installation. Attaching the issue: #2143 |
can i take up this issue?? |
can i work on the issue? |
I'm submitting a ...
Current behaviour:
Expected behaviour:
Steps to reproduce:
Screenshots
Would you like to work on the issue?
Yes.
The text was updated successfully, but these errors were encountered: