Skip to content

CC-167 Add card liability #108

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 20 commits into from
Feb 6, 2023
Merged

CC-167 Add card liability #108

merged 20 commits into from
Feb 6, 2023

Conversation

GreatG0ose
Copy link
Contributor

No description provided.

Vladislav Lipianin added 18 commits January 24, 2023 15:47
[CC-83] Update deployment configuration. Add generated version. Remov…
e8010e1 
…e properties
[CC-83] Update CHANGELOG.md. Remove unused exception
93839ed
[CC-83] Add gh action
aac0218
[CC-83] Skip tests
593fb2c
[CC-83] Update publish config
a297469
[CC-83] Replace env variable with actual value
223cdec
[CC-83] Remove unused files. Remove debug trigger for action
02b174d
[CC-83] Setup hook
1962102
[CC-167] CC-167 Add liablity
770fd34
[CC-167] Disable test
be81af8
[CC-167] Update CHANGELOG.md
a0e243d
[CC-167] Merge remote-tracking branch 'origin/develop' into CC-167/ad…
21aa4f3 
…d-card-liability

# Conflicts:
#	CHANGELOG.md
[CC-167] Update CHANGELOG.md
4d97d9e
[CC-167] Update CHANGELOG.md
53d4c90
[CC-167] Add mock test
769e6f8
[CC-167] Add license header
5753ae9
[CC-167] Fix charge after authorization test
c22dbc0
[CC-167] Fix config test
83a64fe
Ryouzanpaku
Ryouzanpaku reviewed Feb 1, 2023
View reviewed changes
Ryouzanpaku
Ryouzanpaku previously approved these changes Feb 3, 2023
View reviewed changes
Copy link
Contributor

@Ryouzanpaku Ryouzanpaku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm.
I had one comment, though.

[CC-167] Update CHANGELOG.md. Un-deprecate CardTransactionData constr…
a24f550 
…uctor. Added test logging. Enriched logging
@GreatG0ose GreatG0ose marked this pull request as ready for review February 6, 2023 15:57
@GreatG0ose GreatG0ose requested a review from a team as a code owner February 6, 2023 15:57
@GreatG0ose GreatG0ose merged commit 61fa44c into develop Feb 6, 2023
@GreatG0ose GreatG0ose deleted the CC-167/add-card-liability branch February 6, 2023 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ryouzanpaku Ryouzanpaku Ryouzanpaku approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GreatG0ose @Ryouzanpaku