Skip to content

Batch order review#2728

Merged
liamls merged 3 commits intomasterfrom
hotfix/organization-in-payload
Feb 5, 2026
Merged

Batch order review#2728
liamls merged 3 commits intomasterfrom
hotfix/organization-in-payload

Conversation

@liamls
Copy link
Contributor

@liamls liamls commented Feb 4, 2026

During the review of batch orders feature in preproduction environment, we noticed that organization id was missing in the payload in POST requests so we enhanced the existing test and fix the payload.

  • Use richie select component and verify payload in batch order create request.

We also uncheck the default payment method in batch order process, since it's not relevant for users and can create missclicks.

@liamls liamls force-pushed the hotfix/organization-in-payload branch from 43332c9 to 0dbba09 Compare February 4, 2026 14:48
@liamls liamls force-pushed the hotfix/organization-in-payload branch 4 times, most recently from 43a1903 to 5155196 Compare February 4, 2026 15:48
@liamls liamls changed the title 🐛(frontend) add missing organization id in batch order payload Add missing organization id in batch order payload Feb 4, 2026
@liamls liamls force-pushed the hotfix/organization-in-payload branch 2 times, most recently from 5af1946 to f49b8c7 Compare February 5, 2026 09:43
During the review of batch orders feature
in preproduction environment, we noticed
that organization id was missing in the
payload in POST requests so we enhanced
the existing test and fix the payload.
@liamls liamls force-pushed the hotfix/organization-in-payload branch from f49b8c7 to 3fd651f Compare February 5, 2026 10:09
We want to be sure that the user access to the 5th step
in the b2b sale tunnel so we uncheck the default payment
method. It will also avoid miss clicks
@liamls liamls changed the title Add missing organization id in batch order payload Batch order review Feb 5, 2026
We need to update hardcoded user jwt tokens because
they are deprecated since feb 2026.
Copy link
Member

@jonathanreveille jonathanreveille left a comment

Choose a reason for hiding this comment

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

paf

@liamls liamls merged commit cdca96a into master Feb 5, 2026
20 checks passed
@liamls liamls deleted the hotfix/organization-in-payload branch February 5, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants