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

Feature/oxap 292 refactor company handling #117

Merged
merged 2 commits into from
Oct 10, 2019

Conversation

b3nl
Copy link
Contributor

@b3nl b3nl commented Oct 10, 2019

No description provided.

@b3nl b3nl force-pushed the feature/OXAP-292-refactor-company-handling branch from fe8b8dd to 78f6f12 Compare October 10, 2019 07:58
@b3nl b3nl force-pushed the features/make-street-handling-easier branch from 2d0469e to b29f5ce Compare October 10, 2019 10:16
@b3nl b3nl force-pushed the feature/OXAP-292-refactor-company-handling branch 3 times, most recently from 00cb391 to b683fc9 Compare October 10, 2019 10:33
@b3nl b3nl changed the base branch from features/make-street-handling-easier to develop October 10, 2019 10:34
Bjoern Lange added 2 commits October 10, 2019 12:38
…readable

Signed-off-by: Bjoern Lange <bjoern.lange@bestit-online.de>
…g handling for the street number of amazon

Signed-off-by: Bjoern Lange <bjoern.lange@bestit-online.de>
@b3nl b3nl force-pushed the feature/OXAP-292-refactor-company-handling branch from b683fc9 to 301b6e6 Compare October 10, 2019 10:38
@b3nl
Copy link
Contributor Author

b3nl commented Oct 10, 2019

Old Tests run still! New tests are added to map the original pseudo code of amazon and amazon itself says, that it looks good, so i will merge. But a small warning: This are text parsers. There could be another combination of strings which are not mapped yet. We simple can not match every possible combination with this data.

@b3nl b3nl merged commit 65ab614 into develop Oct 10, 2019
@b3nl b3nl deleted the feature/OXAP-292-refactor-company-handling branch October 10, 2019 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant