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 Github Pages Deploy Flow #534

Merged
merged 5 commits into from
Dec 11, 2023
Merged

Update Github Pages Deploy Flow #534

merged 5 commits into from
Dec 11, 2023

Conversation

ymekuria
Copy link
Collaborator

@ymekuria ymekuria commented Dec 8, 2023

Description

This PR updates the Github Pages deploy flow configuration to be compatible with the current version of NextJS. Previously, an error occurred when a user ran npm run deploy to deploy a UI project to Github Pages which blocked all users from deploying a UI with this command.

This occurred because the command in the zkApp-CLI utilizes Next export which has been deprecated and replaced with additional configuration in next.config.js file. The necessary configuration is now added when a user selects the option to deploy to Github Pages in the UI project generation flow.

Addresses #533

Testing

The improvements introduced in this PR have been by tested by generating and deploying UI projects with the zkapp-CLI and running the full e2e testing suite.

… for the Next.js app built files

This property is used to specify the output directory for the Next.js app. By adding this property, we can configure the app to export the built files to a specific directory, which can be useful for deployment purposes.
@ymekuria ymekuria merged commit 1d734bf into main Dec 11, 2023
12 checks passed
@ymekuria ymekuria mentioned this pull request Dec 11, 2023
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.

3 participants