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.

Sign up for GitHub

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

Jump to bottom

Restructure folders and Dockerfiles/compose #234

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

Restructure folders and Dockerfiles/compose #234

wants to merge 1 commit into from

Conversation

darpham
Copy link
Contributor

@darpham darpham commented Oct 8, 2021

  • Move all frontend files to frontend folder
  • Create Dockerfile.cloud for AWS deploys
  • Add Github Action deploys

@darpham darpham requested a review from rwalek668 October 8, 2021 04:19
@darpham
Copy link
Contributor Author

darpham commented Oct 8, 2021 •
edited

@rwalek668
Before merging, I'm having an issue with the API created from redis_api on this branch
It's not correctly fetching data from Redis, so I reverted to previous deploy

@rwalek668
Copy link
Collaborator

Ack I will give this a try to see if I can get it to work. IIRC I was able to get this one working locally before but will double check.

rwalek668
rwalek668 approved these changes Oct 8, 2021
View reviewed changes
Copy link
Collaborator

@rwalek668 rwalek668 left a comment

Choose a reason for hiding this comment

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

The code changes look good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rwalek668 rwalek668 rwalek668 approved these changes

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@darpham @rwalek668