Skip to content

Conversation

justinko
Copy link
Contributor

@justinko justinko commented May 31, 2024

For #51930

@zzak zzak changed the title Update CHANGELOG.md Add changelog for Action Cable TestCookieJar#[]= May 31, 2024
@@ -1,3 +1,21 @@
* Bring `ActionCable::Connection::TestCookieJar` in alignment with `ActionDispatch::Cookies::CookieJar` in regards to setting the cookie value.
Copy link
Member

@zzak zzak May 31, 2024

Choose a reason for hiding this comment

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

Suggested change
* Bring `ActionCable::Connection::TestCookieJar` in alignment with `ActionDispatch::Cookies::CookieJar` in regards to setting the cookie value.
## Unreleased ##
* Bring `ActionCable::Connection::TestCookieJar` in alignment with `ActionDispatch::Cookies::CookieJar` in regards to setting the cookie value.

Might need to add a heading here, waiting on CI (or you can run bundle exec tools/railspect changelogs . locally to verify)

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

We don't use that heading. The header is added on the release

@rafaelfranca rafaelfranca merged commit 46f0dd0 into rails:7-2-stable May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants