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

setting up dev env for admin #296

Open
wants to merge 16 commits into
base: dev
Choose a base branch
from
Open

setting up dev env for admin #296

wants to merge 16 commits into from

Conversation

byronwang93
Copy link
Contributor

@byronwang93 byronwang93 commented Jan 2, 2025

Description

Other considerations

@byronwang93 byronwang93 marked this pull request as ready for review January 2, 2025 08:31
Copy link

github-actions bot commented Jan 2, 2025

Visit the preview URL for this PR (updated for commit 0be0260):

https://nw-admin-portal--pr296-bywang-dev-env-rdoeid6s.web.app

(expires Sun, 19 Jan 2025 05:03:21 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d819f9eb3b1b79c03e9109f08eeff92215a2990e

@byronwang93 byronwang93 changed the title setting up dev env for admin no review yet pls - setting up dev env for admin Jan 2, 2025
@byronwang93 byronwang93 requested a review from DonaldKLee January 4, 2025 00:18
@byronwang93 byronwang93 changed the title no review yet pls - setting up dev env for admin setting up dev env for admin Jan 4, 2025
Comment on lines +11 to +21
env:
IS_PRODUCTION: ${{ endsWith(github.ref, 'main') }}
NODE_OPTIONS: --openssl-legacy-provider
steps:
- uses: actions/checkout@v3
- name: Use Node.js
uses: actions/setup-node@v3
with:
node-version: '16.x'
node-version: '18.x'
- name: Set up Firebase CLI
run: yarn global add firebase-tools
Copy link
Contributor Author

Choose a reason for hiding this comment

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

referenced how we did it in portal

  • node v 16 is not compatible so upgraded to 18

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