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

Move images in public folder into "images" in src folder #948

Closed
mariobehling opened this issue Dec 8, 2017 · 12 comments
Closed

Move images in public folder into "images" in src folder #948

mariobehling opened this issue Dec 8, 2017 · 12 comments
Labels

Comments

@mariobehling
Copy link
Member

All source files including images should be in the src folder.

@Remorax
Copy link
Contributor

Remorax commented Dec 8, 2017

I would like to take this up :)

@Remorax
Copy link
Contributor

Remorax commented Dec 8, 2017

Oh, I didn't notice your comment. I was in the process of creating a PR myself :P

@anshumanv
Copy link
Member

@Remorax will you be sending another PR or can I send one? 😄

@Remorax
Copy link
Contributor

Remorax commented Dec 8, 2017

Sure, go ahead @anshumanv :)

@anshumanv
Copy link
Member

Alright, thanks @Remorax 👍

@anshumanv
Copy link
Member

@mariobehling the scaffolding tool we use is create-react-app which has some restrictions attached. They expect all of the static files(which are copied directly while bundling the app) like favicon etc in the public folder and once we run npm run build to package a production build these files are directly copied to the build folder that is generated. So moving favicon to src/images isn't possible with the current version of CRA. For reference please see - facebook/create-react-app#585

@abishekvashok
Copy link
Member

abishekvashok commented Dec 19, 2017

@anshumanv we could have pre compile script that we should run before npm install in the app.json which copies images to public folder for us and at last cleans up the folder, what do you think?

@anshumanv
Copy link
Member

We can always go for such workarounds, but I'd suggest sticking to the one suggested by cra.

@abishekvashok
Copy link
Member

cra.

???

@anshumanv
Copy link
Member

anshumanv commented Dec 25, 2017

create-react-app.

@abishekvashok
Copy link
Member

@mariobehling status on this

@0x48piraj
Copy link
Member

I can work on this, the other way. (Workaround)

@fragm3 fragm3 closed this as completed May 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants