-
Notifications
You must be signed in to change notification settings - Fork 1
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
Setup new deployment and pr preview deployments #21
Conversation
c489509
to
b4a4bcd
Compare
🚀 Preview deployment for this PR is going up! Remember you might need to hard refresh to get the new content. Preview URL: https://www.frequency.xyz/_pr/21/ |
c2486bd
to
2ce0969
Compare
- name: Upload artifact | ||
uses: actions/upload-pages-artifact@v3 | ||
- name: Deploy to GitHub Pages | ||
uses: JamesIves/github-pages-deploy-action@881db5376404c5c8d621010bcbec0310b58d5e29 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JamesIves is the sort of "standard" old way to deploy github pages. We have the commit sha there instead of a version as we allowlist non-github actions.
- name: Install Node.js | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: 20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's the difference between say this is "20.x"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is the same
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of questions, but LGTM
closes #13 ## Changes - Added footer content - Added layout styling so that the footer is always at the bottom. ## Demo <img width="1266" alt="Screenshot 2024-02-13 at 8 08 17 PM" src="https://github.com/AmplicaLabs/frequency-xyz/assets/43625033/2b414d5b-394c-4614-99f9-8caa4c5cac92"> <img width="431" alt="Screenshot 2024-02-13 at 8 08 11 PM" src="https://github.com/AmplicaLabs/frequency-xyz/assets/43625033/9e4f6f97-a52e-423e-8d33-c95c904c9405"> --------- Co-authored-by: Claire Olmstead <olmsteadclaire@gmail.com> Co-authored-by: Wil Wade <wil.wade@amplica.io>
Description
As a non-technical (or technical) reviewer of site changes, I want to see the site running so that I can see how it actually works.
Screenshot(s)
Type of change
How to Test?
Please describe how to test that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration
Checklist: