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

Allow different form attributes for new/update actions #1991

Merged
merged 5 commits into from
Jun 24, 2021

Conversation

kolasss
Copy link
Contributor

@kolasss kolasss commented May 25, 2021

No description provided.

@kolasss
Copy link
Contributor Author

kolasss commented May 26, 2021

where is .rubocop.yml? i can't review all errors locally?

Copy link
Collaborator

@pablobm pablobm left a comment

Choose a reason for hiding this comment

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

Thank you for the PR @kolasss! I think it's pretty good. If you can just have a look at my comments, I think we can get this merged soon.

Regarding your question about the Rubocop config, I think Hound uses the rules found at https://github.com/thoughtbot/guides/blob/main/ruby/.rubocop.yml

lib/administrate/base_dashboard.rb Outdated Show resolved Hide resolved
lib/administrate/page/form.rb Outdated Show resolved Hide resolved
Co-authored-by: Pablo Brasero <36066+pablobm@users.noreply.github.com>
Copy link
Collaborator

@pablobm pablobm left a comment

Choose a reason for hiding this comment

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

Perfect. Thank you!

@pablobm pablobm merged commit 5eccfd4 into thoughtbot:main Jun 24, 2021
@alesrosina
Copy link

This thing looks like I would need - is there any plan on making next gem release? Current 0.16.0 does not yet include this change.

jordan-brough pushed a commit to rinsed-org/administrate that referenced this pull request Jan 21, 2022
Co-authored-by: Pablo Brasero <36066+pablobm@users.noreply.github.com>
jordan-brough added a commit to rinsed-org/administrate that referenced this pull request Jan 24, 2022
…) (#24)

Co-authored-by: Pablo Brasero <36066+pablobm@users.noreply.github.com>

Co-authored-by: Nikolai Ammosov <kolas.batman@gmail.com>
Co-authored-by: Pablo Brasero <36066+pablobm@users.noreply.github.com>
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