Skip to content

Provide customer name and email to Checkout#131

Merged
ocasta181 merged 20 commits intodevelopfrom
task/andrin/str-433
Mar 3, 2023
Merged

Provide customer name and email to Checkout#131
ocasta181 merged 20 commits intodevelopfrom
task/andrin/str-433

Conversation

@ocasta181
Copy link
Copy Markdown
Contributor

@ocasta181 ocasta181 commented Mar 1, 2023 •
edited
Loading

  1. Create a new user
  2. Give that user a name
  3. Create a transaction
  4. Check the transaction in the checkout.com dashboard (https://dashboard.sandbox.checkout.com/payments)
  5. Confirm the transaction reflects the name provided with the card, the user's name, and the users email

frostbournesb
frostbournesb reviewed Mar 2, 2023
View reviewed changes
@ocasta181 ocasta181 changed the title Add string-user name to checkout authentication Provide customer name and email to Checkout Mar 2, 2023
akfoster and others added 10 commits March 2, 2023 14:43
@frostbournesb
Update pkg/model/transaction.go
c1a5722 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/repository/auth.go
654b9aa 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/internal/common/receipt.go
ae1b24a 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/service/checkout.go
72b2d28 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update migrations/0003_contact-to-platform_device-to-instrument_tx-le…
6f7c9a4 
…g_transaction.sql

Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/internal/common/sign.go
077e60d 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/internal/common/sign.go
8eb1527 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/repository/device.go
29b3beb 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/internal/common/receipt.go
9a4addc 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
@frostbournesb
Update pkg/internal/common/receipt.go
504b9da 
Co-authored-by: Frostbourne <85953565+frostbournesb@users.noreply.github.com>
ocasta181
ocasta181 commented Mar 2, 2023
View reviewed changes
@alcalawil
Copy link
Copy Markdown
Contributor

alcalawil commented Mar 2, 2023

What am I doing wrong?
image

alcalawil
alcalawil previously approved these changes Mar 2, 2023
View reviewed changes
saito-sv
saito-sv reviewed Mar 2, 2023
View reviewed changes
@saito-sv
Update pkg/repository/auth.go
7600a88 
Co-authored-by: saito-sv <7920256+saito-sv@users.noreply.github.com>
@saito-sv
Update pkg/internal/unit21/evaluate_test.go
c1ad8bb 
Co-authored-by: saito-sv <7920256+saito-sv@users.noreply.github.com>
@ocasta181 ocasta181 requested review from alcalawil and saito-sv March 3, 2023 00:02
@ocasta181 ocasta181 merged commit 0506665 into develop Mar 3, 2023
@ocasta181 ocasta181 deleted the task/andrin/str-433 branch March 3, 2023 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@auroter auroter Awaiting requested review from auroter

3 more reviewers

@frostbournesb frostbournesb frostbournesb left review comments

@saito-sv saito-sv saito-sv approved these changes

@alcalawil alcalawil alcalawil approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ocasta181 @alcalawil @saito-sv @frostbournesb