Skip to content

Fix setting request body plain text in transaction#538

Merged
MiSikora merged 3 commits into
developfrom
plain-text-request
Jan 29, 2021
Merged

Fix setting request body plain text in transaction#538
MiSikora merged 3 commits into
developfrom
plain-text-request

Conversation

@MiSikora

Copy link
Copy Markdown
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

@cortinico cortinico left a comment

Copy link
Copy Markdown
Member

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.

3 participants