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.

Sign up for GitHub

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

Compatibility with rails_admin > 3.1 and rails > 6.1 #30

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Compatibility with rails_admin > 3.1 and rails > 6.1 #30

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
4b04504
Convert views from .html.halm to .html.erb
oloomoses Mar 3, 2023
f7f610b
Fix table tags in history.html.erb
oloomoses Mar 6, 2023
11c84d9
Add jquery widgets
oloomoses Mar 8, 2023
bdf594e
Add rails_admin.dom_ready
oloomoses Mar 9, 2023
4cde396
Fix modal error
oloomoses Apr 4, 2023
fcbe017
Add main.js entry poing and fix dependency issues
oloomoses Jun 2, 2023
32c347a
Add auditing_limit_version
oloomoses Dec 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,14 @@ You may also need to `require "paper_trail"` in `application.rb`

As pointed out in issue [#8](https://github.com/rikkipitt/rails_admin_history_rollback/issues/8), `paper_trail` master branch and release [v5.0.0](https://github.com/airblade/paper_trail/blob/master/CHANGELOG.md#500-unreleased) (when available) causes session issues. Login is reset by devise; `paper_trail` tries to access to the `current_user` variable before `devise` ([airblade/paper_trail#556](https://github.com/airblade/paper_trail#556)).

The fix is to manually add `set_paper_trail_whodunnit` before_action as advised here: (https://github.com/airblade/paper_trail#finding-out-who-was-responsible-for-a-change)
The fix is to manually add `set_paper_trail_whodunnit` before_action as advised here: (https://github.com/airblade/paper_trail#finding-out-who-was-responsible-for-a-change)

Paper trial `changeset` returns a nil value, and therefor the popup widget will be black. To fix this add

```
config.active_record.use_yaml_unsafe_load = true
```
in `config/application.rb` as discussed in here: (https://stackoverflow.com/questions/72970170/upgrading-to-rails-6-1-6-1-causes-psychdisallowedclass-tried-to-load-unspecif)

```ruby
# app/controllers/application_controller.rb
Expand Down
Loading