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

upgrade activeadmin #674

Merged
merged 1 commit into from
Apr 17, 2020
Merged

Conversation

Ivanov-Anton
Copy link
Collaborator

@Ivanov-Anton Ivanov-Anton commented Apr 15, 2020

commit upgrade activeadmin
upgraded activeadmin from 1.4.3 to 2.7.0

casue of: outputting following warning
"DEPRECATION WARNING: Active Admin: The register_stylesheet config is deprecated and will be removed in v2." link

"DEPRECATION WARNING: Active Admin: The type parameter has been deprecated. Provide the "true" type as a class instead." link

removed parameter named type from all methods called status_tag

@Ivanov-Anton Ivanov-Anton changed the title upgrade activeadmin WIP: upgrade activeadmin Apr 15, 2020
@Ivanov-Anton Ivanov-Anton force-pushed the upgrade_activeadmin branch 2 times, most recently from 4ca8428 to af97db3 Compare April 16, 2020 09:55
@Ivanov-Anton Ivanov-Anton changed the title WIP: upgrade activeadmin upgrade activeadmin Apr 16, 2020
end
row('Silenly drop') do |r|
status_tag(r.silently_drop.to_s, r.silently_drop.to_s, class: r.silently_drop? ? :ok : :red)
status_tag(r.silently_drop.to_s, staclass: r.silently_drop? ? :ok : :red)
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: staclass

@gigorok gigorok merged commit 19cf3d3 into yeti-switch:master Apr 17, 2020
@Ivanov-Anton Ivanov-Anton deleted the upgrade_activeadmin branch May 14, 2020 13:30
@Ivanov-Anton
Copy link
Collaborator Author

Also solves issue number #570

@Fivell
Copy link
Contributor

Fivell commented Feb 17, 2021

@Ivanov-Anton seems it has regressions #885

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.

3 participants