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

Added multiple database Active Record guide entry to the release notes #36823

Conversation

abhaynikam
Copy link
Contributor

@abhaynikam abhaynikam commented Jul 31, 2019

ref: #36820 (comment)

Added the entry in the release notes for new guide.

@rails-bot rails-bot bot added the docs label Jul 31, 2019
@abhaynikam abhaynikam force-pushed the add-multiple-database-guide-entry-to-release-notes branch from b7eb13e to 4c7ba2c Compare July 31, 2019 13:07
@bogdanvlviv
Copy link
Contributor

This should go to the master first and then we backport to the 6-0-stable.

@abhaynikam
Copy link
Contributor Author

Oh. I didn't knew that. Let me fix it 😊

@abhaynikam abhaynikam force-pushed the add-multiple-database-guide-entry-to-release-notes branch from 4c7ba2c to 5828aad Compare July 31, 2019 13:11
@abhaynikam abhaynikam changed the base branch from 6-0-stable to master July 31, 2019 13:12
@abhaynikam
Copy link
Contributor Author

I am sorry about the labels, it happened because I changed the base to 6-0-stable to master. This is docs only change.

@@ -1001,6 +1001,8 @@ Ruby on Rails Guides
Please refer to the [Changelog][guides] for detailed changes.

### Notable changes
* Add Multiple Databases with Active Record guide.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add an empty line between "Notable changes" and this line for consistency with other sections?

@abhaynikam abhaynikam force-pushed the add-multiple-database-guide-entry-to-release-notes branch from 5828aad to 0db87b9 Compare August 1, 2019 14:30
Copy link
Contributor

@bogdanvlviv bogdanvlviv left a comment

Choose a reason for hiding this comment

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

Thank you!

@eileencodes eileencodes merged commit 170f511 into rails:master Aug 5, 2019
eileencodes added a commit that referenced this pull request Aug 5, 2019
…-entry-to-release-notes

Added multiple database Active Record guide entry to the release notes
@abhaynikam abhaynikam deleted the add-multiple-database-guide-entry-to-release-notes branch August 5, 2019 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants