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

Update local environment setup #285

Merged
merged 3 commits into from
Mar 19, 2020

Conversation

ariariel
Copy link
Contributor

@ariariel ariariel commented Mar 19, 2020

While trying to follow the project README.md to set up my local environment there were a few hiccups that I first had to address in the solution. This may be good to have updated to make it a bit smoother for the next person to onboard. Although there is probably more left to configure I believe this is the bare minimum that needs to be addressed in order to get the app up and running locally. If there are other obvious things that can be updated in the README perhaps this is a good time to do so?

Command: bundle exec rake dummy:posts
Error: NameError: uninitialized constant Faker::Books::HitchhikersGuideToTheGalaxy
Resolution: After looking at Faker documentation, I see that this one does exist: Faker::Movies::HitchhikersGuideToTheGalaxy.quote (Movies instead of Books). So I changed Books to Movies to fix it locally

Command: react-scripts start
Error: TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
Resolution: I found facebook/create-react-app#8499 and also https://stackoverflow.com/questions/60234640/typeerror-err-invalid-arg-type-the-path-argument-must-be-of-type-string-re
So I bumped up react-scripts version 3.3.0 --> 3.4.0 to fix it locally.

Command: rails s
Error: bucket.rb:684:in 'extract_name': missing required option :name (ArgumentError)
Resolution: added S3_BUCKET my environment variables, and modified README to denote it as required.

@ariariel ariariel changed the title Optimizations/518 update local env setup Update local environment setup Mar 19, 2020
@ryandeshon ryandeshon merged commit e8319c8 into development Mar 19, 2020
@anaulin anaulin deleted the optimizations/518-update-local-env-setup branch May 7, 2021 23:38
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