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

Fixed bug where checkout to location would throw an error if FMCS was enabled #13655

Merged
merged 2 commits into from
Sep 26, 2023

Conversation

snipe
Copy link
Owner

@snipe snipe commented Sep 26, 2023

If full multiple company support is enabled AND the asset has a company ID AND the asset is checked out to location (which doesn't have a company ID), the checkout would fail with a mismatch error.

@shortcut-integration
Copy link

@snipe snipe merged commit 1509339 into develop Sep 26, 2023
@snipe snipe deleted the bug/sc-23759 branch September 26, 2023 13:37
@what-the-diff
Copy link

what-the-diff bot commented Sep 26, 2023

PR Summary

  • Improved Validation in Asset Checkout Process
    A condition has been added for the asset checkout process in our system. This checks if the ID of the company associated with the target and the asset is not null before checking if they match. If they don't match, it redirects users to the checkout page again with an error message. This enhancement will provide better data integrity and user experience.

  • Updated Error Message Details
    The error message that gets displayed when the user and asset companies do not match has been made more specific and clear. Now it accurately reflects that the checkout target company and asset company do not match. This change will make error messages more easily understandable for users.

@snipe
Copy link
Owner Author

snipe commented Sep 26, 2023

(This fixes a regression created by #13400)

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