added fax and mobile mapping path#1
Merged
eiriarte-mendez merged 1 commit intoratepay:masterfrom Feb 24, 2020
Merged
Conversation
Contributor
|
Hi Chris, Best regards |
eiriarte-mendez
pushed a commit
that referenced
this pull request
Jul 22, 2019
…ckend_orders to master * commit '93dcddba2c4b0874be21e239f525219135299bf3': feature/SHPWR-137_backend_orders move util method from shopware to lib
Contributor
eiriarte-mendez
left a comment
eiriarte-mendez
left a comment
There was a problem hiding this comment.
Found it finally into the src. Sorry for delay
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
I'm working for DeineTuer GmbH in Leipzig and we're using your RatePay-Modul for Magento 1.9
In the last two days there were multiple payment attemps returning 'Model exception : Model 'Fax' invalid '.
We figured out, the sub model name path mapping for fax and mobile was missing.
The fix has been successfully tested and deployed in our stating and live system.
Kind regards,
Chris