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

5675.1 pass requisition id on insert inbound #6247

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

roxy-dao
Copy link
Contributor

Part 1 of #5675

πŸ‘©πŸ»β€πŸ’» What does this PR do?

Attempting to break down the issue for easier review. This PR allows endpoint to pass in requisition_id and should throw an error if you try to link requisition to inbound shipment without manually_link_internal_order_to_inbound_shipment store preference.

πŸ§ͺ Testing

  • Don't turn on manually_link_internal_order_to_inbound_shipment store pref
  • Use graphiql to create a new inbound shipment and link to an internal order by providing requisition_id
  • Should have error
  • Now turn on pref and try again
  • Should be okay to link

πŸ“ƒ Documentation

  • Part of an epic: documentation will be completed for the feature as a whole
  • No documentation required: no user facing changes or a bug fix which isn't a change in behaviour
  • These areas should be updated or checked:
    1.
    2.

@github-actions github-actions bot added this to the v2.6.0 milestone Jan 22, 2025
@github-actions github-actions bot added enhancement New feature or request Team Ruru πŸ¦‰ Roxy, Ferg, Noel feature: CIV labels Jan 22, 2025
Copy link

Bundle size difference

Comparing this PR to main

Old size New size Diff
5.17 MB 5.17 MB 85 B (0.00%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature: CIV Team Ruru πŸ¦‰ Roxy, Ferg, Noel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant