Skip to content

Conversation

@RaoHai
Copy link
Contributor

@RaoHai RaoHai commented Sep 24, 2024

  • 纯静态部署

@vercel
Copy link

vercel bot commented Sep 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
petercat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 4:14pm

@petercat-assistant
Copy link

Walkthrough

This pull request introduces the deployment configuration for deploying static content to S3. It includes changes to the GitHub Actions workflow and updates to the Next.js configuration to support static export.

Changes

File Summary
.github/workflows/aws-static.yml Added a new GitHub Actions workflow for deploying static content to S3.
client/next.config.js Updated Next.js configuration to support static export when NEXT_STANDALONE environment variable is set.

- .github/workflows/aws-static.yml
- client/**
- assistant/**

Choose a reason for hiding this comment

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

Ensure that the AWS credentials used here are securely stored and not hard-coded. Consider using GitHub Secrets for sensitive information.

Copy link
Contributor

Choose a reason for hiding this comment

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

展开说说


const nextConfig = {
...process.env.NEXT_STANDALONE ? { output: "standalone" } :{},
...process.env.NEXT_STANDALONE ? { output: "export", ssr: false } :{},

Choose a reason for hiding this comment

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

Setting ssr: false might have unintended side effects if your application relies on server-side rendering. Ensure this change is thoroughly tested.

@codecov
Copy link

codecov bot commented Sep 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

@RaoHai
Copy link
Contributor Author

RaoHai commented Sep 24, 2024

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