Skip to content

Refactor the devise_error_messages! helper to render a partial#4616

Merged
tegon merged 1 commit intoheartcombo:masterfrom
prograhamer:refactor-error-view-to-template
Dec 18, 2018
Merged

Refactor the devise_error_messages! helper to render a partial#4616
tegon merged 1 commit intoheartcombo:masterfrom
prograhamer:refactor-error-view-to-template

Conversation

@prograhamer
Copy link
Copy Markdown
Contributor

This should make it more intuitive and easier for developers using the project to implement their own layout for errors.

This should make it more intuitive and easier for developers using the
project to implement their own layout for errors.
@tegon
Copy link
Copy Markdown
Member

tegon commented Nov 28, 2018

It seems ok to me. @rafaelfranca Do you have any concerns about this?

@tegon tegon merged commit 76b87dc into heartcombo:master Dec 18, 2018
afomera pushed a commit to afomera/devise-bootstrapped that referenced this pull request Feb 15, 2019
* Update readme with Bootstrap 4 information

* Revised bundler gem to version greater than 1.12

* Render partial instead of devise_error_messages!

This will remove the deprecation warning from DeviseHelper#devise_error_messages message as from Devise 4.6.0 in favor of rendering partials. Ref: #5, heartcombo/devise#4616, and platformatec/devise@76b87dc0e83736cf16e3ffbc465fcd8ee3c06d46

* Add email_changed.html.erb

* Bump version

* Fixed Travis error
ionut998 added a commit to ionut998/activeadmin that referenced this pull request Mar 1, 2019
From version 4.6 Devise deprecated the use of `devise_error_messages!` helper method in favour of rendering a the `error_messages` partial.

[Devise PR](heartcombo/devise#4616) that introduced this.

Warning message that get's solved with this PR
```
DEPRECATION WARNING: [Devise] `DeviseHelper.devise_error_messages!` is deprecated and it will be removed in the next major version.
To customize the errors styles please run `rails g devise:views` and modify the `devise/shared/error_messages` partial.
```

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants