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

Fix linter dependency and webpack container #10

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

afreydev
Copy link
Contributor

@afreydev afreydev commented Jun 29, 2023

Hotfix PR

This PR fix webpack container rebuilding node-saas dependency during the container start. Additionally, this fixes the babel dependency error.

Upgrading webpack container node version to 14.

Status

  • Urgent (ready to deploy ASAP)
  • Non-urgent (ready, deploying in next release is ok)
  • Awaiting feedback
  • WIP
  • Already deployed (awaiting post-review)

QA Checklist

  • Build is passing
  • Playtested (up-to-date with master) all pages that touch code that changed

Normal PR

[describe changes and/or link to related issues, post screenshots. If you are waiting for feedback describe it here]

Status

  • Urgent (ready to deploy ASAP)
  • Non-urgent (ready, deploying in next release is ok)
  • awaiting feedback
  • WIP

QA Checklist

(copy/paste in the relevant checklist. Check anything that you did, and leave it unchecked if you didn't. It's ok to leave things unchecked if you don't think they matter to your PR)

Frontend Changes

  • Build is passing
  • Was playtested when up-to-date with current dev
  • Component has been playtested locally
    • followed the playtesting guide as necessary
    • including logged in/out
    • including dark / light
    • including different device sizes
    • including all major browsers
  • Component has been playtested on beta
    • followed the playtesting guide as necessary
    • including logged in/out
    • including dark / light
    • including different device sizes
    • including all major browsers
    • including all gametypes

Backend Changes

  • New test cases were written that failed before the change was made
  • Build is passing
  • Was playtested when up-to-date with current dev
  • All relevant components are working correctly in local playtesting
    • including logged in/out
    • including other relevant state changes [e.g. with an account that lacks a buy-in. please list them]
  • Performance seems ok in beta playtesting
    • including any relevant state changes [e.g. in a bounty hand. please list them]

@afreydev afreydev force-pushed the afreydev/fix-webpack-container branch from 5da145c to 3072149 Compare June 29, 2023 17:23
@afreydev afreydev merged commit 0c63889 into main Jun 29, 2023
2 checks passed
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.

1 participant