- Issue created by @rszrama
- πΊπΈUnited States rszrama
This will need to be backported to 8.x-1.x as well.
- Status changed to Needs review
8 months ago 9:52pm 4 April 2024 - last update
8 months ago 8 pass - last update
8 months ago 8 pass - πΊπΈUnited States rszrama
Here are the patches, one for each branch. I do wish we could actually surface the exception message in the UI, but ultimately I think we may just need some form enhancement to make it clear that the number would always be required and the file is optional. Not really sure of the intent behind the current form, to be honest.
- First commit to issue fork.
- Merge request !11Issue #3427116 by rszrama: Throw a decline exception instead of an invalid... β (Merged) created by fathershawn
-
FatherShawn β
committed ac053fc7 on 8.x-1.x
Issue #3427116 by rszrama: Throw a decline exception instead of an...
-
FatherShawn β
committed ac053fc7 on 8.x-1.x
-
FatherShawn β
committed 6f295b3f on 2.0.x
Issue #3427116: Throw a decline exception instead of an invalid argument...
-
FatherShawn β
committed 6f295b3f on 2.0.x
- Status changed to Fixed
8 months ago 11:44am 5 April 2024 - πΊπΈUnited States fathershawn New York
Tagged a 1.4 release on the unsupported 8.x-1.x branch
I'll tag a full 2.0 release after marking this issue fixed.
Thanks for the collaboration @rszrama! My original client required a PO number. After this was contributed, a feature was contributed for the file upload of a copy of the PO. The instructions should be configured for the user in the gateway config so that the tasks are clear.
Automatically closed - issue fixed for 2 weeks with no activity.