Skip to content

Conversation

JKE-be
Copy link
Contributor

@JKE-be JKE-be commented Mar 14, 2025

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by google_account, which manages tokens and requests to Google.

This also fixes the call to get_param('google_calendar_client_secret'), which should use the correct helper _get_client_secret, as it may be monkey-patched by another module.

…r sync

Before this commit, the `refresh_token` handling was done in the `calendar`
module,   even though it relates to token management and should therefore be
handled by `google_account`, which manages tokens and requests to Google.

This also fixes the call to `get_param('google_calendar_client_secret')`,
which should use the correct helper `_get_client_secret`, as it may be
monkey-patched by another module.
@robodoo
Copy link
Contributor

robodoo commented Mar 14, 2025

Pull request status dashboard

@JKE-be JKE-be requested review from dalaOdoo and geflx March 14, 2025 08:13
@C3POdoo C3POdoo requested review from a team March 14, 2025 08:15
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 14, 2025
@arj-odoo
Copy link
Contributor

@robodoo r+

@robodoo robodoo closed this in a8e0aec Mar 17, 2025
@fw-bot fw-bot deleted the 18.0-calendar-3-jke branch March 24, 2025 20:45
omar-sherif9992 pushed a commit to odoo-dev/odoo that referenced this pull request Mar 26, 2025
…r sync

Before this commit, the `refresh_token` handling was done in the `calendar`
module,   even though it relates to token management and should therefore be
handled by `google_account`, which manages tokens and requests to Google.

This also fixes the call to `get_param('google_calendar_client_secret')`,
which should use the correct helper `_get_client_secret`, as it may be
monkey-patched by another module.

closes odoo#201702

Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Mar 27, 2025
…r sync

Before this commit, the `refresh_token` handling was done in the `calendar`
module,   even though it relates to token management and should therefore be
handled by `google_account`, which manages tokens and requests to Google.

This also fixes the call to `get_param('google_calendar_client_secret')`,
which should use the correct helper `_get_client_secret`, as it may be
monkey-patched by another module.

closes odoo/odoo#201702

Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
arj-odoo pushed a commit to odoo-dev/odoo that referenced this pull request Apr 2, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also odoo#201702

task-4653852
robodoo pushed a commit that referenced this pull request Apr 3, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also #201702

task-4653852

closes #202309

Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
nikj-odoo added a commit to odoo-dev/odoo that referenced this pull request Apr 3, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also odoo#201702

task-4653852

X-original-commit: 84998fa
robodoo pushed a commit that referenced this pull request Apr 4, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also #201702

task-4653852

closes #204619

X-original-commit: 84998fa
Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
fw-bot pushed a commit to odoo-dev/odoo that referenced this pull request Apr 4, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also odoo#201702

task-4653852

X-original-commit: 06f0add
fw-bot pushed a commit to odoo-dev/odoo that referenced this pull request Apr 4, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also odoo#201702

task-4653852

X-original-commit: 06f0add
robodoo pushed a commit that referenced this pull request Apr 7, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also #201702

task-4653852

closes #204897

X-original-commit: 06f0add
Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
Signed-off-by: Nikhil Joshi (nikj) <nikj@odoo.com>
robodoo pushed a commit that referenced this pull request Apr 7, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also #201702

task-4653852

closes #204899

X-original-commit: 06f0add
Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
Signed-off-by: Nikhil Joshi (nikj) <nikj@odoo.com>
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Apr 26, 2025
…r calendar

Before this commit, the refresh_token handling was done in the calendar
module, even though it relates to token management and should therefore be
handled by microsoft_account, which manages tokens and requests to Google.

See also odoo/odoo#201702

task-4653852

closes odoo/odoo#202309

Signed-off-by: Arnaud Joset (arj) <arj@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants