Skip to content

Conversation

@kushthedude
Copy link
Member

Fixes #3454

@kushthedude
Copy link
Member Author

@shreyanshdwivedi @uds5501 Please review

stripeClientId : attr('string'),
stripeSecretKey : attr('string'),
stripePublishableKey : attr('string'),
stripeTestCliendId : attr('string'),
Copy link
Member

Choose a reason for hiding this comment

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

Why this passed in review?

Copy link
Member

Choose a reason for hiding this comment

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

@iamareebjamal Typo mistake.

Copy link
Member Author

Choose a reason for hiding this comment

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

Why this passed in review?

Might the PR be of large size, Thats why we would have missed noticing the error.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why does anything pass in review indeed 😢

stripeClientId : attr('string'),
stripeSecretKey : attr('string'),
stripePublishableKey : attr('string'),
stripeTestCliendId : attr('string'),
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does anything pass in review indeed 😢

@abhinavk96 abhinavk96 merged commit def4113 into fossasia:development Aug 21, 2019
@kushthedude kushthedude deleted the str branch November 8, 2019 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stripe Client ID is not being saved

5 participants