Skip to content
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

Feature: Sign out of the extension #116

Closed
2 tasks done
diivi opened this issue May 24, 2023 · 3 comments · Fixed by #138
Closed
2 tasks done

Feature: Sign out of the extension #116

diivi opened this issue May 24, 2023 · 3 comments · Fixed by #138

Comments

@diivi
Copy link
Contributor

diivi commented May 24, 2023

Type of feature

🍕 Feature

Current behavior

There's no way to sign out directly through the popup.

Suggested solution

A simple signout button on the profile page should be sufficient. It can clear the cookie that we use, but this will sign the user out from insights.opensauced too.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs
@Anush008
Copy link
Member

Anush008 commented May 24, 2023

To note, the extension logs you in automatically when installed, on browser start-up. The log-out feature has to handle this too.

chrome.runtime.onInstalled.addListener(checkAuthentication);
chrome.runtime.onInstalled.addListener(setDefaultDescriptionConfig);
chrome.runtime.onStartup.addListener(checkAuthentication);
.

Currently, the way to log-out of the extension is to logout from https://insights.opensauced.pizza.

@github-actions
Copy link

🎉 This issue has been resolved in version 1.3.0-beta.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This issue has been resolved in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants