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

Allow gchat login page #403

Merged
merged 3 commits into from
Aug 23, 2023
Merged

Allow gchat login page #403

merged 3 commits into from
Aug 23, 2023

Conversation

iconeb
Copy link
Contributor

@iconeb iconeb commented Aug 17, 2023

Pre-flight Checklist

Please ensure you've completed all of the following.

Description of Change

Add reference to workspace.google.com to redirect to login page, similar to other google services like Calendar and Mail

@SpecialAro SpecialAro requested a review from a team August 23, 2023 14:57
Copy link
Member

@SpecialAro SpecialAro left a comment

Choose a reason for hiding this comment

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

Tested and LGTM!

@iconeb, would you please bump the package.json version to 1.8.0 ? This way, users will be notified to upgrade to the new recipe version.

Thank you very much!

@SpecialAro
Copy link
Member

SpecialAro commented Aug 23, 2023

Also, I think this PR closes ferdium/ferdium-app#1323

@SpecialAro SpecialAro linked an issue Aug 23, 2023 that may be closed by this pull request
2 tasks
Copy link
Member

@SpecialAro SpecialAro left a comment

Choose a reason for hiding this comment

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

Bumped the version myself to approve and merge 😄

@SpecialAro
Copy link
Member

@vraravam looks like the build is failing but the version bump is ok. I dont see any errors on my local when running pnpm package... Should we merge?

image

@vraravam
Copy link
Contributor

yes, this kind of flakiness has occurred in the past. lets merge

iconeb and others added 2 commits August 23, 2023 15:52
Add reference to workspace.google.com to redirect to login page
@vraravam
Copy link
Contributor

vraravam commented Aug 23, 2023

Linter found some issues - I have added a new commit to this PR

@iconeb - please follow instructions to ensure that the PR is complete and correct.

Thanks for your contribution

@SpecialAro SpecialAro merged commit ee5695b into ferdium:main Aug 23, 2023
2 checks passed
@vraravam
Copy link
Contributor

@all-contributors - please add @iconeb for code

@allcontributors
Copy link
Contributor

@vraravam

I've put up a pull request to add @iconeb! 🎉

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.

Google Hangouts Chat unable to signin
3 participants