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

feat: transaction confirmation and verification flow #127

Merged
merged 34 commits into from
Feb 27, 2022

Conversation

zainfathoni
Copy link
Owner

@zainfathoni zainfathoni commented Feb 26, 2022

Closes #75

@codecov-commenter
Copy link

codecov-commenter commented Feb 26, 2022

Codecov Report

Merging #127 (c1095a6) into main (8a2d8c1) will decrease coverage by 1.14%.
The diff coverage is 4.71%.

❗ Current head c1095a6 differs from pull request most recent head 49040c3. Consider uploading reports for the commit 49040c3 to get more accurate results

Impacted file tree graph

@@            Coverage Diff            @@
##             main    #127      +/-   ##
=========================================
- Coverage   10.69%   9.54%   -1.15%     
=========================================
  Files          56      63       +7     
  Lines         589     838     +249     
  Branches      179     293     +114     
=========================================
+ Hits           63      80      +17     
- Misses        525     757     +232     
  Partials        1       1              
Impacted Files Coverage Δ
app/components/button-link.tsx 0.00% <0.00%> (ø)
app/components/form-elements.tsx 0.00% <0.00%> (ø)
app/components/page-link.tsx 0.00% <0.00%> (ø)
app/components/sections/pricing.tsx 0.00% <0.00%> (ø)
app/components/transaction-details.tsx 0.00% <0.00%> (ø)
app/models/enum.ts 100.00% <ø> (ø)
app/models/subscription.ts 0.00% <0.00%> (ø)
app/models/transaction.ts 0.00% <0.00%> (ø)
app/routes/dashboard.tsx 0.00% <ø> (ø)
app/routes/dashboard/index.tsx 0.00% <0.00%> (ø)
... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a2d8c1...49040c3. Read the comment docs.

@zainfathoni zainfathoni force-pushed the feature/verification-flow branch from 525d8f5 to a009f97 Compare February 26, 2022 18:37
@zainfathoni zainfathoni changed the title feat: verification flow feat: transaction confirmation and verification flow Feb 27, 2022
@zainfathoni zainfathoni marked this pull request as ready for review February 27, 2022 17:33
kodiakhq[bot]
kodiakhq bot previously approved these changes Feb 27, 2022
@zainfathoni zainfathoni added automerge To be merged automatically once all the requirements are fulfilled and removed automerge To be merged automatically once all the requirements are fulfilled labels Feb 27, 2022
kodiakhq[bot]
kodiakhq bot previously approved these changes Feb 27, 2022
@zainfathoni zainfathoni added the automerge To be merged automatically once all the requirements are fulfilled label Feb 27, 2022
@kodiakhq kodiakhq bot merged commit 4cb11b4 into main Feb 27, 2022
@kodiakhq kodiakhq bot deleted the feature/verification-flow branch February 27, 2022 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge To be merged automatically once all the requirements are fulfilled
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Payment Confirmation
2 participants