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

Update the INSTALL.md file to include steps for deploying al-folio on Netlify. #2798

Merged
merged 4 commits into from
Dec 4, 2024

Conversation

HenryF23
Copy link
Contributor

@HenryF23 HenryF23 commented Oct 22, 2024

Update CUSTOMIZE.md to include steps to deploy al-folio on Netlify.

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for relaxed-lollipop-b6bc17 ready!

Name Link
🔨 Latest commit b3c594c
🔍 Latest deploy log https://app.netlify.com/sites/relaxed-lollipop-b6bc17/deploys/674fa1df53d6d000088101f1
😎 Deploy Preview https://deploy-preview-2798--relaxed-lollipop-b6bc17.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 configuration.

@HenryF23 HenryF23 mentioned this pull request Oct 22, 2024
2 tasks
@HenryF23 HenryF23 changed the title update CUSTOMIZE.md to include steps to deploy al-folio on Netlify Update the CUSTOMIZE.md file to include steps for deploying al-folio on Netlify. Oct 22, 2024
Copy link
Collaborator

@george-gca george-gca left a comment

Choose a reason for hiding this comment

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

Sorry for the delay on this. Can you move this to INSTALL.md? More specifically, above Deployment to another hosting server (non GitHub Pages). I think it makes more sense there. Also, 2 more changes:

  1. change branch master to main, since we are not using master anymore
  2. make it as an enumeration. I think it is easier to follow

@HenryF23
Copy link
Contributor Author

HenryF23 commented Dec 3, 2024

Sorry for the delay on this. Can you move this to INSTALL.md? More specifically, above Deployment to another hosting server (non GitHub Pages). I think it makes more sense there. Also, 2 more changes:

  1. change branch master to main, since we are not using master anymore
  2. make it as an enumeration. I think it is easier to follow

I have moved the documentation to INSTALL.md and used enumeration for better readability. The branch should already be main

INSTALL.md Outdated Show resolved Hide resolved
@HenryF23 HenryF23 changed the title Update the CUSTOMIZE.md file to include steps for deploying al-folio on Netlify. Update the INSTALL.md file to include steps for deploying al-folio on Netlify. Dec 4, 2024
@george-gca george-gca merged commit a340b60 into alshedivat:main Dec 4, 2024
6 checks passed
@george-gca
Copy link
Collaborator

Thanks for your contribution.

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.

2 participants