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

Fixes qty remaining requirements for component checkout via API #13728

Merged
merged 2 commits into from
Oct 9, 2023

Conversation

snipe
Copy link
Owner

@snipe snipe commented Oct 9, 2023

This retreads the fix from @Singrity in #13725 that was mis-targeted. It allows you to checkout components with the same number of available items as you have available. (I believe this was fixed in the UI a while back but not in the API)

@snipe snipe changed the title Fixes/13725 for dev Fixes qty remaining requirements for component checkout via API Oct 9, 2023
@what-the-diff
Copy link

what-the-diff bot commented Oct 9, 2023

PR Summary

  • New Contributor Introduction
    The system files have been updated to introduce Bogdan, a new contributor to the project. All relevant details, including their login details, name, and profile information have been added.

  • Project Contributors Update
    The main project documentation has been updated to reflect the new number of contributors, which has increased from 326 to 327. This incorporates the newest member, Bogdan.

  • Component Checkout Code Improvement
    Improvements have been made in the code that controls the way we handle components within the system. We have addressed a condition in our checkout functionality, ensuring it correctly verifies the assigned quantity against the number of available components. This will make the checkout process smoother and more accurate.

@snipe snipe merged commit e3591dc into develop Oct 9, 2023
6 checks passed
@snipe snipe deleted the fixes/13725_for_dev branch October 9, 2023 15:40
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.

1 participant