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

chore: fix route html #232

Merged
merged 2 commits into from Nov 26, 2020
Merged

chore: fix route html #232

merged 2 commits into from Nov 26, 2020

Conversation

@xrkffgg
Copy link
Member

@xrkffgg xrkffgg commented Nov 26, 2020

https://field-form.react-component.now.sh/demo/basic

链接为具体 demo 时,会 404

https://field-form.react-component.now.sh/ 主页,选 Demo 不会

@vercel
Copy link

@vercel vercel bot commented Nov 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/field-form/jbn13uydw
Preview: https://field-form-git-config.react-component.vercel.app

@xrkffgg
Copy link
Member Author

@xrkffgg xrkffgg commented Nov 26, 2020

好像不是这个原因。。。

@PeachScript help

@codecov
Copy link

@codecov codecov bot commented Nov 26, 2020

Codecov Report

Merging #232 (bee269a) into master (ffb84c0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #232   +/-   ##
=======================================
  Coverage   99.89%   99.89%           
=======================================
  Files          13       13           
  Lines         954      954           
  Branches      222      222           
=======================================
  Hits          953      953           
  Misses          1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffb84c0...bee269a. Read the comment docs.

.umirc.ts Outdated
@@ -7,5 +7,6 @@ export default defineConfig({
'https://avatars0.githubusercontent.com/u/9441414?s=200&v=4',
logo:
'https://avatars0.githubusercontent.com/u/9441414?s=200&v=4',
outputPath: '.doc'
outputPath: '.doc',
ssr: {}

This comment has been minimized.

@PeachScript

PeachScript Nov 26, 2020

开启 exportStatic 会按照路由输出 HTML

Suggested change
ssr: {}
ssr: {},
exportStatic: {}

另外 now.json 应该要配置下 routes:https://github.com/umijs/dumi/blob/master/now.json#L12

This comment has been minimized.

@xrkffgg

xrkffgg Nov 26, 2020
Author Member

好的,我试试

@xrkffgg xrkffgg changed the title chore: add ssr chore: fix route html Nov 26, 2020
@xrkffgg
Copy link
Member Author

@xrkffgg xrkffgg commented Nov 26, 2020

可以了~ 感谢 @PeachScript

@xrkffgg xrkffgg merged commit 2c5c382 into master Nov 26, 2020
9 checks passed
9 checks passed
setup
Details
lint
Details
compile
Details
coverage
Details
LGTM analysis: JavaScript No new or fixed alerts
Details
Vercel Deployment has completed
Details
WIP Ready for review
Details
codecov/patch Coverage not affected when comparing ffb84c0...bee269a
Details
codecov/project 99.89% (+0.00%) compared to ffb84c0
Details
@delete-merged-branch delete-merged-branch bot deleted the config branch Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.