Skip to content

[FIX] session: always return a Promise#7725

Closed
fw-bot wants to merge 1 commit intosaas-19.1from
saas-19.1-19.0-transport-promise-lul-488433-fw
Closed

[FIX] session: always return a Promise#7725
fw-bot wants to merge 1 commit intosaas-19.1from
saas-19.1-19.0-transport-promise-lul-488433-fw

Conversation

@fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Jan 7, 2026

If the implementation of transportService doesn't return a Promise when calling sendMessage, the session breaks.

With this commit, we ensure we always get a Promise which allows to safely call .then or .catch

Task: 0

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

review checklist

  • feature is organized in plugin, or UI components
  • support of duplicate sheet (deep copy)
  • in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • in model/UI: ranges are strings (to show the user)
  • undo-able commands (uses this.history.update)
  • multiuser-able commands (has inverse commands and transformations where needed)
  • new/updated/removed commands are documented
  • exportable in excel
  • translations (_t("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

Forward-Port-Of: #7721

If the implementation of `transportService` doesn't
return a Promise when calling `sendMessage`, the session breaks.

With this commit, we ensure we always get a Promise which allows
to safely call `.then` or `.catch`

Task: 0
X-original-commit: b5f5d6d
@robodoo
Copy link
Collaborator

robodoo commented Jan 7, 2026

Pull request status dashboard

@fw-bot
Copy link
Collaborator Author

fw-bot commented Jan 7, 2026

This PR targets saas-19.1 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot fw-bot mentioned this pull request Jan 7, 2026
14 tasks
robodoo pushed a commit that referenced this pull request Jan 7, 2026
If the implementation of `transportService` doesn't
return a Promise when calling `sendMessage`, the session breaks.

With this commit, we ensure we always get a Promise which allows
to safely call `.then` or `.catch`

closes #7725

Task: 0
X-original-commit: b5f5d6d
Signed-off-by: Vincent Schippefilt (vsc) <[email protected]>
@robodoo robodoo closed this Jan 7, 2026
@fw-bot fw-bot deleted the saas-19.1-19.0-transport-promise-lul-488433-fw branch January 14, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants