-
Notifications
You must be signed in to change notification settings - Fork 191
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
Conversation
There was a problem hiding this 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!
Also, I think this PR closes ferdium/ferdium-app#1323 |
a8b3105
to
44f6ca1
Compare
There was a problem hiding this 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 😄
@vraravam looks like the build is failing but the version bump is ok. I dont see any errors on my local when running |
yes, this kind of flakiness has occurred in the past. lets merge |
Add reference to workspace.google.com to redirect to login page
263bdc7
to
4e90406
Compare
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 |
@all-contributors - please add @iconeb for code |
I've put up a pull request to add @iconeb! 🎉 |
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