customerio / customerio-android

This is the official Customer.io SDK for Android.
MIT License
13 stars 9 forks source link

fix: remove tasks from queue with 400 http response #177

Closed Shahroz16 closed 1 year ago

Shahroz16 commented 1 year ago

closes: https://github.com/customerio/issues/issues/9443

Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

github-actions[bot] commented 1 year ago

Pull request title looks good 👍!

If this pull request gets merged, it will cause a new release of the software. Example: If this project's latest release version is 1.0.0. If this pull request gets merged in, the next release of this project will be 1.0.1. This pull request is not a breaking change.

All merged pull requests will eventually get deployed. But some types of pull requests will trigger a deployment (such as features and bug fixes) while some pull requests will wait to get deployed until a later time.

This project uses a special format for pull requests titles. Expand this section to learn more (expand by clicking the ᐅ symbol on the left side of this sentence)...
This project uses a special format for pull requests titles. Don't worry, it's easy! This pull request title should be in this format: ``` : short description of change being made ``` **If your pull request [introduces breaking changes](https://web.archive.org/web/20220725195319/https://nordicapis.com/what-are-breaking-changes-and-how-do-you-avoid-them/)** to the code, use this format: ``` !: short description of breaking change ``` where `` is one of the following: - `feat:` - A feature is being added or modified by this pull request. Use this if you made any changes to any of the features of the project. - `fix:` - A bug is being fixed by this pull request. Use this if you made any fixes to bugs in the project. - `docs:` - This pull request is making documentation changes, only. - `refactor:` - A change was made that doesn't fix a bug or add a feature. - `test:` - Adds missing tests or fixes broken tests. - `style:` - Changes that do not effect the code (whitespace, linting, formatting, semi-colons, etc) - `perf:` - Changes improve performance of the code. - `build:` - Changes to the build system (maven, npm, gulp, etc) - `ci:` - Changes to the CI build system (Travis, GitHub Actions, Circle, etc) - `chore:` - Other changes to project that don't modify source code or test files. - `revert:` - Reverts a previous commit that was made. ### Examples: ``` feat: edit profile photo refactor!: remove deprecated v1 endpoints build: update npm dependencies style: run formatter ``` Need more examples? Want to learn more about this format? [Check out the official docs](https://www.conventionalcommits.org/). **Note:** If your pull request does multiple things such as adding a feature _and_ makes changes to the CI server _and_ fixes some bugs then you might want to consider splitting this pull request up into multiple smaller pull requests.
github-actions[bot] commented 1 year ago

Build available to test Version: shahroz-remove-400-requests-from-queue-SNAPSHOT Repository: https://s01.oss.sonatype.org/content/repositories/snapshots/

codecov[bot] commented 1 year ago

Codecov Report

Merging #177 (b16fec0) into main (d1aad3a) will increase coverage by 1.50%. The diff coverage is 71.42%.

@@             Coverage Diff              @@
##               main     #177      +/-   ##
============================================
+ Coverage     62.02%   63.52%   +1.50%     
- Complexity      210      218       +8     
============================================
  Files            91       91              
  Lines          2041     2051      +10     
  Branches        258      263       +5     
============================================
+ Hits           1266     1303      +37     
+ Misses          678      646      -32     
- Partials         97      102       +5     
Impacted Files Coverage Δ
...ain/java/io/customer/sdk/data/store/FileStorage.kt 67.50% <28.57%> (-4.73%) :arrow_down:
...main/java/io/customer/sdk/api/HttpRequestRunner.kt 75.00% <50.00%> (-2.20%) :arrow_down:
...main/java/io/customer/sdk/queue/QueueRunRequest.kt 90.38% <93.75%> (+1.25%) :arrow_up:
...o/customer/sdk/error/CustomerIOApiErrorResponse.kt 100.00% <100.00%> (ø)
...main/java/io/customer/sdk/error/CustomerIOError.kt 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.