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 setting request body plain text in transaction #538

Merged
merged 3 commits into from
Jan 29, 2021

Conversation

MiSikora
Copy link
Contributor

πŸ“„ Context

There is a bug while processing requests.

πŸ“ Changes

I set a correct property on a transaction. Previously it used response, now it uses request.

πŸ“Ž Related PR

#527

🚫 Breaking

No.

πŸ› οΈ How to test

I added a test that covers the bug.

@MiSikora MiSikora added the bug Something isn't working label Jan 27, 2021
@cortinico cortinico added this to the 4.0.0 milestone Jan 28, 2021
Copy link
Member

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

Nice catch πŸ‘Œ

@MiSikora MiSikora merged commit ac2db2f into develop Jan 29, 2021
@MiSikora MiSikora deleted the plain-text-request branch January 29, 2021 14:13
vbuberen pushed a commit that referenced this pull request Jun 26, 2021
* Fix setting request body plain text in transaction

* Add test for plain text request body
vbuberen added a commit that referenced this pull request Jun 28, 2021
* Update all dependencies and configs, update code for Detekt

* Prepare for next release (#494)

* Prepare for next release
* Update workflows and readme to mirror latest changes

* Update publishing action name (#495)

* Fixed typo on DialogData (#501)

* Update README.md (#505)

fix the typo

* Fix setting request body plain text in transaction (#538)

* Fix setting request body plain text in transaction

* Add test for plain text request body

* Switch to CircularProgressIndicator

* Switch to Activity Result API

* Add immutable flag to pending intents (#593)

* Update Github Actions workflows to match latest ones

* Bump version

* Resolve lint issues

* Remove test using newer OkHttp API

* Remove breaking change with BuildConfig removal

* Bump kotlinVersion from 1.5.10 to 1.5.20 (#639)

Bumps `kotlinVersion` from 1.5.10 to 1.5.20.

Updates `kotlin-gradle-plugin` from 1.5.10 to 1.5.20
- [Release notes](https://github.com/JetBrains/kotlin/releases)
- [Changelog](https://github.com/JetBrains/kotlin/blob/v1.5.20/ChangeLog.md)
- [Commits](JetBrains/kotlin@v1.5.10...v1.5.20)

Updates `kotlin-stdlib` from 1.5.10 to 1.5.20
- [Release notes](https://github.com/JetBrains/kotlin/releases)
- [Changelog](https://github.com/JetBrains/kotlin/blob/v1.5.20/ChangeLog.md)
- [Commits](JetBrains/kotlin@v1.5.10...v1.5.20)

---
updated-dependencies:
- dependency-name: org.jetbrains.kotlin:kotlin-gradle-plugin
  dependency-type: direct:production
  update-type: version-update:semver-patch
- dependency-name: org.jetbrains.kotlin:kotlin-stdlib
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: Nicola Corti <corti.nico@gmail.com>
Co-authored-by: Okan AYDIN <okanaydin1994@gmail.com>
Co-authored-by: MichaΕ‚ Sikora <michalsikora90@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants