Skip to content

Conversation

@chukitow
Copy link
Contributor

@chukitow chukitow commented Nov 6, 2017

On Rails 5.1 form_tag & form_for were unified in form_with, I'm adding the helper when using Rails >= 5.1

@chukitow chukitow closed this Nov 6, 2017
@bootstrap-ruby-bot
Copy link

bootstrap-ruby-bot commented Nov 6, 2017

2 Warnings
⚠️ There are code changes, but no corresponding tests. Please include tests if this PR introduces any modifications in rails-bootstrap-forms’s behavior.
⚠️ Please update CHANGELOG.md with a description of your changes. If this PR is not a user-facing change (e.g. just refactoring), you can disregard this.

Here's an example of a CHANGELOG.md entry (place it immediately under the * Your contribution here! line):

* [#354](https://github.com/bootstrap-ruby/rails-bootstrap-forms/pull/354): Add bootstrap form with helper on rails 5.1 - [@chukitow](https://github.com/chukitow).

Generated by 🚫 Danger

@chukitow chukitow reopened this Nov 6, 2017
@mattbrictson mattbrictson added this to the 4.0.0 milestone Jan 6, 2018
@lcreid
Copy link
Contributor

lcreid commented Jan 8, 2018

PR #346 adds support for form_with with extensive testing. [Disclaimer: I submitted that PR.] It now has merge conflicts which I'll resolve once we get a few of the other pending PRs merged into master.

@lcreid
Copy link
Contributor

lcreid commented Jan 9, 2018

PR #346 was replaced by PR #369.

@lcreid
Copy link
Contributor

lcreid commented Jan 9, 2018

@mattbrictson , could you please review this one, as I have an obvious conflict of interest. :-) If I understand correctly, this PR was submitted against master back when master was still Bootstrap 3.

@mattbrictson
Copy link
Contributor

Closed in favor of #369.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants