Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: r6stats #50

Merged
merged 1 commit into from
Oct 15, 2022
Merged

fix: r6stats #50

merged 1 commit into from
Oct 15, 2022

Conversation

G0maa
Copy link
Contributor

@G0maa G0maa commented Oct 15, 2022

Description

Just re-arranged dockerfile, it's more of directed for a production build currently.

You can run it the usual way in the documentation or using docker compose up, I didn't try deploying it though.

Mostly I think the problem was in the COPY in dockerfile.

Fixes #5

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Tested on my docker, I think @KieranRobson have to try it on his computer first & testing deployment.

Checklist:

  • My code follows the style guidelines of this project
  • I have restored .env.template
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes - Run npm run format
  • Any dependent changes have been merged and published in downstream modules

@netlify
Copy link

netlify bot commented Oct 15, 2022

Deploy Preview for superb-froyo-91afbf ready!

Name Link
🔨 Latest commit 35f4b8b
🔍 Latest deploy log https://app.netlify.com/sites/superb-froyo-91afbf/deploys/634a80d38bba27000899fd50
😎 Deploy Preview https://deploy-preview-50--superb-froyo-91afbf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@Ki-er
Copy link
Owner

Ki-er commented Oct 15, 2022

We can give it ago and revert if it doesnt work!

@Ki-er Ki-er merged commit 9643e14 into Ki-er:master Oct 15, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/r6stats - Doesnt work on deployed docker
2 participants