Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Fix 496#497

Open
championpaddler wants to merge 2 commits into
fossasia:masterfrom
championpaddler:Fix-496
Open

Fix 496#497
championpaddler wants to merge 2 commits into
fossasia:masterfrom
championpaddler:Fix-496

Conversation

@championpaddler

Copy link
Copy Markdown
Member

Fixes #496

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream master branch.
  • The unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • All the functions created/modified in this PR contain relevant docstrings.

Changes proposed in this pull request:

  • Alerts on changing passwords.

Comment thread src/js/options.js Outdated
}, (stream) => {
stream.getTracks().forEach(track => track.stop());
displayNotification('success', ' Mic Access Granted', 1000);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the extra blank line

Comment thread src/js/options.js Outdated
localStorage.setItem("theme",theme);
alert("Success");
});
displayNotification('info', 'Theme successfully changed', 1000);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please make the indentation consistent?

@the-codingninja the-codingninja left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do the requested changes

@rajvaibhavdubey rajvaibhavdubey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the conflicts, else LGTM 👍

@championpaddler

Copy link
Copy Markdown
Member Author

@ms10398 Done Now.

@championpaddler

Copy link
Copy Markdown
Member Author

@ms10398 @stealthanthrax Please review.

@championpaddler

Copy link
Copy Markdown
Member Author

@ms10398 @mariobehling Please review.

@samagragupta samagragupta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@championpaddler

Copy link
Copy Markdown
Member Author

@hkedia321 Please approve this also.

@championpaddler

Copy link
Copy Markdown
Member Author

@gabru-md @DNS-404 Please review.

@fragm3 fragm3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try to combine the changes done in #495 with this, looks almost same @sk9331657

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No alerts in case password is wrong while resetting password

5 participants