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

[FIX] website_payment: remove action return #162382

Closed

Conversation

vchu-odoo
Copy link
Contributor

@vchu-odoo vchu-odoo commented Apr 18, 2024

When you click 'Activate Stripe' button it threw a traceback.
Reason: It seems like returning server action does not work anymore.

@vchu-odoo vchu-odoo marked this pull request as ready for review April 18, 2024 09:34
@robodoo
Copy link
Contributor

robodoo commented Apr 18, 2024

@vchu-odoo vchu-odoo force-pushed the saas-17.2-fix-stripe-action-vchu branch 2 times, most recently from 9c8b0ec to 300f4eb Compare April 18, 2024 11:21
@vchu-odoo vchu-odoo changed the title [FIX] website_payment: convert button to action type [FIX] website_payment: remove action return Apr 18, 2024
When you click 'Activate Stripe' button it threw a traceback.
Reason: It seems like returning server action does not work anymore.
@vchu-odoo vchu-odoo force-pushed the saas-17.2-fix-stripe-action-vchu branch from 300f4eb to e8707fa Compare April 18, 2024 11:27
Copy link
Contributor

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

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

@robodoo r+
@fw-bot ignore

@fw-bot
Copy link
Contributor

fw-bot commented Apr 18, 2024

Forward-port disabled.

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 18, 2024
robodoo pushed a commit that referenced this pull request Apr 18, 2024
When you click 'Activate Stripe' button it threw a traceback.

closes #162382

Reason: It seems like returning server action does not work anymore.
Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@robodoo robodoo closed this Apr 18, 2024
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.

None yet

5 participants