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

show status on receipts #468

Merged
merged 3 commits into from
May 23, 2023
Merged

show status on receipts #468

merged 3 commits into from
May 23, 2023

Conversation

fremartini
Copy link
Member

closes #446

@fremartini fremartini requested a review from a team May 23, 2023 16:28
@fremartini fremartini enabled auto-merge (squash) May 23, 2023 16:29
@ghost
Copy link

ghost commented May 23, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Merging #468 (cb76a4a) into develop (aafdb56) will increase coverage by 0.06%.
The diff coverage is 73.33%.

@@             Coverage Diff             @@
##           develop     #468      +/-   ##
===========================================
+ Coverage    66.03%   66.10%   +0.06%     
===========================================
  Files           99      100       +1     
  Lines         1107     1118      +11     
===========================================
+ Hits           731      739       +8     
- Misses         376      379       +3     
Flag Coverage Δ
unittests 66.10% <73.33%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...es/receipt/data/models/purchase_receipt_model.dart 0.00% <0.00%> (ø)
...ures/receipt/domain/entities/purchase_receipt.dart 33.33% <0.00%> (ø)
lib/features/receipt/domain/entities/receipt.dart 100.00% <ø> (ø)
...tures/purchase/domain/entities/payment_status.dart 61.53% <85.71%> (+28.20%) ⬆️
lib/core/extensions/string_extensions.dart 100.00% <100.00%> (ø)
...ta/repositories/opening_hours_repository_impl.dart 89.28% <100.00%> (ø)
...ipt/data/repositories/receipt_repository_impl.dart 100.00% <100.00%> (ø)

TTA777
TTA777 previously approved these changes May 23, 2023
Copy link
Member

@TTA777 TTA777 left a comment

Choose a reason for hiding this comment

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

lgtm

@fremartini fremartini merged commit 7070c6f into develop May 23, 2023
@fremartini fremartini deleted the frem/purchase_status branch May 23, 2023 18:09
@marfavi marfavi mentioned this pull request May 28, 2023
marfavi added a commit that referenced this pull request May 30, 2023
# New features
* show status on receipts (#468)
* show timestamp on receipt list entry (#478)

# Fixes
* Fix showing raw html in error messages (#463)
* fix(tickets): Fix multiple `CoffeeCard`s sharing the same product id (#471)
* fix: Avoid type casting 204 return types from API (#479)
* fix(receipts): Remove purchase receipt view & refactor payment status display (#478)
* Added awaits for useTicket and FetchReceipts (#464)

# Code refactoring and added tests
* test(utils): Improve String.capitalize() and add tests (#472)
* Refactor and test Leaderboard (#443)
* Refactor payment (#465)
* Refactor environment logic (#469)
* refactor(settings): tidy up `settings_page.dart` (#473)
* Restructure settings (#481)
* Refactor and test login (#483)
* refactor and test voucher (#480)
* Refactor and test register (#486)
* Refactor and test products (#482)

# Chores
* chore: update Bloc packages (#475)
* chore: bump flutter required version to 3.10.2 (#476)
* Migrate to Dart 3 and Flutter 3.10.0 (#452)

# Developer related
* migrate to fpdart (#457)
* Install the CodeSee workflow (#467)
* Update README.md to show coverage on develop branch (#484)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App shows purchase as Purchased when it is not in a completed state
2 participants