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
#57 We can specify a custom card type logos #63
#57 We can specify a custom card type logos #63
Conversation
example/lib/main.dart
Outdated
@@ -77,6 +77,17 @@ class MySampleState extends State<MySample> { | |||
useBackgroundImage ? 'assets/card_bg.png' : null, | |||
isSwipeGestureEnabled: true, | |||
onCreditCardWidgetChange: (CreditCardBrand creditCardBrand) {}, | |||
cardType: CardType.mastercard, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the requirement of cardtype?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its optional parameter not required, I Removed
CHANGELOG.md
Outdated
@@ -1,3 +1,6 @@ | |||
## 3.0.1 - Sep 28, 2021 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please mention this as unreleased : [Unreleased]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added [Unreleased] tag
lib/credit_card_widget.dart
Outdated
} | ||
|
||
return icon; | ||
} | ||
|
||
List<CustomCardTypeImage> getCustomCardTypeIcon(CardType currentCardType) { | ||
final List<CustomCardTypeImage> customCardTypeIcon = widget.customCardIcons |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No requirement to use variable, you can directly return the value
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
❇️ fix #57 We Can specify a custom card type logos