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

fix: Remove null type cast in receipt_remote_data_source.dart #479

Merged
merged 2 commits into from
May 27, 2023

Conversation

marfavi
Copy link
Member

@marfavi marfavi commented May 26, 2023

Closes #426

@ghost
Copy link

ghost commented May 26, 2023

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

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@marfavi marfavi changed the title fix: Avoid type casting 204 return types from API fix: Remove null type cast in receipt_remote_data_source.dart May 26, 2023
@codecov
Copy link

codecov bot commented May 26, 2023

Codecov Report

Merging #479 (067586e) into develop (82c5bc3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #479   +/-   ##
========================================
  Coverage    67.76%   67.76%           
========================================
  Files          101      101           
  Lines         1123     1120    -3     
========================================
- Hits           761      759    -2     
+ Misses         362      361    -1     
Flag Coverage Δ
unittests 67.76% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
...t/data/datasources/receipt_remote_data_source.dart 90.00% <100.00%> (+5.38%) ⬆️

@fremartini
Copy link
Member

has this been fixed in the backend?

@marfavi
Copy link
Member Author

marfavi commented May 27, 2023

has this been fixed in the backend?

Yes, #426 (comment)

@fremartini fremartini enabled auto-merge (squash) May 27, 2023 09:06
@fremartini fremartini merged commit 75a22dc into develop May 27, 2023
@fremartini fremartini deleted the avoid-null-casting-api-response branch May 27, 2023 09:19
@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.

Avoid type casting 204 return types from API
2 participants