Fix #373 [SuiteCRM 8]Google API Token for calendar sync in User Profi…

[abuzarfaris]

…le not working

Open the Google auth window in a new tab

Fixes salesagility/SuiteCRM-Core#373 in SuiteCRM 8

Description

Motivation and Context

How To Test This

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

• My code follows the code style of this project found here.
• My change requires a change to the documentation.
• I have read the How to Contribute guidelines.

[johnM2401]

Fix LGTM

However, this seems to be pointed at the Suite7 Repo as opposed to the SuiteCRM-Core repo.

Should this be ported over to the hotfix branch on SuiteCRM-Core, @clemente-raposo?

If not, this looks good and can be merged.

Thanks!

[abuzarfaris]

Fix LGTM

However, this seems to be pointed at the Suite7 Repo as opposed to the SuiteCRM-Core repo.

Should this be ported over to the hotfix branch on SuiteCRM-Core, @clemente-raposo?

If not, this looks good and can be merged.

Thanks!

Hey @johnM2401
I was advised to create this Pull request on the SuiteCRM 7 repo to maintain similar code on both repos
Here is a link to that discussion
salesagility/SuiteCRM-Core#374 (comment)

[johnM2401]

Hey @johnM2401 I was advised to create this Pull request on the SuiteCRM 7 repo to maintain similar code on both repos Here is a link to that discussion salesagility/SuiteCRM-Core#374 (comment)

Ahhhh I see, thank you for clarifying!

In that case, looks good to me, i'll approve for merging now.

Thank you!