8000 Update 9-batch-actions.md to reflect new form strategy. by maxkaplan · Pull Request #8744 · activeadmin/activeadmin · GitHub
[go: up one dir, main page]

Skip to content

Update 9-batch-actions.md to reflect new form strategy. #8744

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maxkaplan
Copy link

Hey all, I noticed the docs did not show a valid way to make a batch action modal form. It still shows the old way that used the 'form' param.

I replaced the old form: param content with the upgraded v4 example from : https://my.diffend.io/gems/activeadmin/4.0.0.beta2/4.0.0.beta3

Updated documentation to reflect new batch action modal and removed old `form:` param info. Ref: https://my.diffend.io/gems/activeadmin/4.0.0.beta2/4.0.0.beta3
@tagliala
Copy link
Contributor

Hello, at the moment the documentation covers 3.x series and changes needed for v4 are in the UPGRADING.md guide

Does the new change also work in v3?

@maxkaplan
Copy link
Author
maxkaplan commented Jun 18, 2025

Hello! Ok my mistake, I guess I was confused by the docs UI, I thought the top nav was letting me know these are the docs for version 4.

I have not tested if this approach works in v3.

Again, sorry about my confusion.
I have been using ActiveAdmin for years and really appreciate all the work and wanted to try and help out in some meager way.
Best,

image

@tagliala
Copy link
Contributor

yes, the documentation at the moment is confusing and there is no version switch

So there is still no official documentation for v4, I don't know the plans for that

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.

2 participants
0