Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Improve setup instructions#8545

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
tagliala wants to merge1 commit intomaster
base:master
Choose a base branch
Loading
fromchore/fix-installation-guide

Conversation

tagliala
Copy link
Contributor

  • Separates database setup from server startup and clarifies asset handling.
  • Copycssbundling-rails instructions from Upgrading guide as a recommended approach for managing Tailwind CSS assets
  • Suggest to start the server withbin/dev to handle both web and CSS building processes.

Ref:#8538

[ci skip]

thisismydesign reacted with thumbs up emoji
@taglialataglialaforce-pushed thechore/fix-installation-guide branch 4 times, most recently from024db0e to8b381a9CompareNovember 17, 2024 10:22
- Separate database setup from server startup and clarifies asset  handling process- Copy `cssbundling-rails` instructions from the Upgrading guide as a  recommended approach for managing Tailwind CSS assets- Suggest starting the server with `bin/dev` to handle both web and CSS  building processRef:#8538[ci skip]
@taglialataglialaforce-pushed thechore/fix-installation-guide branch from8b381a9 toae6eebaCompareNovember 17, 2024 10:22
rails importmap:install

bundle add cssbundling-rails
rails css:install:tailwind
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not necessary to install tailwind. One can run commandrails css:install and it should work.

Copy link
Contributor

Choose a reason for hiding this comment

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

I mean, gem 'tailwindcss-rails' may not be installed


```sh
bundle add importmap-rails
rails importmap:install
Copy link
Contributor

Choose a reason for hiding this comment

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

As far as I know from my upgrade experience - it's also not necessary.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

rails importmap:install is not necessary because it is being performed byrails g active_admin:install


```sh
bin/dev
```
Copy link
Contributor

Choose a reason for hiding this comment

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

I think for those who doesn't have Procfile.dev you should give another option:

yarn build:css rails s

@andrewroth
Copy link

Hi,

I'm starting a new rails 8 app with active admin, and the styles didn't load at all, after following the basic guide.

After some searching I came across this, and followed the commands, but I get02:15:57 css.1 | Specified input file ./app/assets/stylesheets/active_admin.css does not exist. -- any ideas?

Should I just go back to rails 7 for now and accept rails 8 and active admin are incompatible (is this actually the case)? Thanks.

@tagliala
Copy link
ContributorAuthor

tagliala commentedNov 26, 2024
edited
Loading

Should I just go back to rails 7 for now and accept rails 8 and active admin are incompatible (is this actually the case)?

They are compatible:

#8538 (reply in thread)

If you have some sort of edge cases, or different instructions, please provide more information so we can improve the installation guide


edit: guide updated and tested again, I confirm that Rails 8.0 and Active Admin 4 beta are compatible

image

@andrewroth
Copy link

I still have no css after hours of effort.

If I runnpx tailwindcss -i ./app/assets/stylesheets/active_admin.css -o ./app/assets/builds/active_admin.css --minify -c tailwind-active_admin.config.js manually, I get:

Rebuilding...Error: Cannot find module 'tailwindcss/plugin'Require stack:... {  code: 'MODULE_NOT_FOUND',  requireStack: [    '/Users/andrew/Development/node_modules/@activeadmin/activeadmin/plugin.js'  ]}

I've googled "Error: Cannot find module 'tailwindcss/plugin'" and tried every possible fix I can find with no success.

@andrewroth
Copy link

Is it possible to use active admin without tailwind?

@tagliala
Copy link
ContributorAuthor

tagliala commentedNov 26, 2024
edited
Loading

Can you please provide the list of commands that you are executing?

Also please include Ruby and Node versions

ruby --version# ruby 3.3.6 (2024-11-05 revision 75015d4c1f) [x86_64-darwin24]node --version# v20.18.1rails _8.0.0_ new my_appcd my_appbundle add activeadmin -v=4.0.0beta14bundle add deviserails g active_admin:installbundle add cssbundling-railsrails css:install:tailwindrails db:migraterails db:seedyarn add @activeadmin/activeadmin@4.0.0-beta14npm pkg set scripts.build:css="tailwindcss -i ./app/assets/stylesheets/active_admin.css -o ./app/assets/builds/active_admin.css --minify -c tailwind-active_admin.config.js"

Is it possible to use active admin without tailwind?

Can't tell, maybe there are infos here:#8223

@andrewroth
Copy link

I am on an old node and osx, maybe that has something to do with it?

node --versionv16.16.0ruby --versionruby 3.3.6 (2024-11-05 revision 75015d4c1f) [x86_64-darwin17]

@tagliala
Copy link
ContributorAuthor

node --version
v16.16.0

This may be an issue, but we are not aware of such issue.

I remember that I couldn't update to v4 with Node 16, but I need to check why

@tagliala
Copy link
ContributorAuthor

I'm successfully running the application with node16.20.2

@andrewroth
Copy link

That did get further, staring from a new app from scratch.

Now I get this:

image

@tagliala
Copy link
ContributorAuthor

looks like assets are not being build

Can you try./bin/dev instead ofrails s and report any error in the asset building?

fabriciofreitag reacted with thumbs up emoji

@renatoaug
Copy link

That did get further, staring from a new app from scratch.

Now I get this:

image

I am with this same problem here.

I ran using./bin/dev and the result was:

image

@tagliala
Copy link
ContributorAuthor

Hi,

These are the instructions to create an app from scratch

SuperAdmin::CompaniesController looks like existing code

@renatoaug
Copy link

Hi,

These are the instructions to create an app from scratch

SuperAdmin::CompaniesController looks like existing code

Hi@tagliala ,

I was talking about the broken layout:

image

I followed all the steps herehttps://github.com/activeadmin/activeadmin/blob/master/UPGRADING.md to upgrade from v3 to v4 but I think the css is not being applied.

@teaforthecat
Copy link

Hi, just want to point out that this is a reasonable alternative to cssbundler if the goals are these:

  • ActiveAdmin 3
  • Rails 8 (with or without tailwind)
  • Not requiring node
bundle add dartsass-railsecho"Rails.application.config.dartsass.builds = {\n  'active_admin.scss' => 'active_admin.css'\n}"> config/initializers/dartsass.rbecho"sass: bin/rails dartsass:watch">> Procfile.devbin/dev

I wonder if this could go into theactiveadmin:install task just for the time between Rails 8 being out and ActiveAdmin 4 not yet out?

teenu4 reacted with thumbs up emoji

@taglialatagliala mentioned this pull requestJan 29, 2025
1 task
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@thisismydesignthisismydesignthisismydesign left review comments

@PoroshkinaVVPoroshkinaVVPoroshkinaVV left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

6 participants
@tagliala@andrewroth@renatoaug@teaforthecat@thisismydesign@PoroshkinaVV

[8]ページ先頭

©2009-2025 Movatter.jp