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

feat: add flag to not open the browser automatically #159

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

alxndr13
Copy link
Contributor

Sometimes i want to authenticate using an incognito window, but calendarsync opens the auth dialog in the last recent window i used. To make sure i can select which browser, window or profile i want to use for the authentication, i can now add the flag --open-browser false to get a link which can be copied to the browser or window the user selects.

@alxndr13 alxndr13 requested a review from MichaelEischer June 16, 2024 12:14
Copy link
Collaborator

@MichaelEischer MichaelEischer left a comment

Choose a reason for hiding this comment

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

There's something missing here, the openBrowser argument is just passed along, but never used.

@alxndr13 alxndr13 force-pushed the open-browser-flag branch from b75744a to 0944848 Compare July 10, 2024 09:36
@alxndr13
Copy link
Contributor Author

@MichaelEischer thanks for the heads up. I had some changes that I hadn't committed yet. They are committed now, please take a look. :)

I was surprised when you mentioned that this variable wasn't used. I've been using my custom build for four weeks without issues, but I guess I need to commit my changes. 😄

@MichaelEischer MichaelEischer merged commit 16a5c4d into main Jul 10, 2024
5 checks passed
@MichaelEischer MichaelEischer deleted the open-browser-flag branch July 10, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants