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

[Refactor]: 重构 render/pages 目录下三个文件, 使用 Vue3 组合式 API 写法 #113

Merged
merged 1 commit into from
May 15, 2024

Conversation

alwayrun
Copy link
Contributor

改动内容

使用 Vue3 组合式 API 写法, 重构下例三个文件
render/pages/ErrorPage.vue
render/pages/IndexPage.vue
render/pages/SuccessPage.vue

@alwayrun alwayrun changed the title refactor: 重构 render/pages 目录下三个文件, 使用 Vue3 组合式 API 写法 [Refactor]: 重构 render/pages 目录下三个文件, 使用 Vue3 组合式 API 写法 May 15, 2024
@@ -28,115 +30,101 @@ import encrypt from '../utils/encrypt'

Copy link
Collaborator

Choose a reason for hiding this comment

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

vscode校验提示:Module '"xiaoju-survey/web/src/render/components/ProgressBar.vue"' has no default export.Vetur

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个应该是 vscode Vetur 插件导致的,我看我这边 vscode 没有这个校验提示

isMobile?: boolean,
}

const props = withDefaults(defineProps<Props>(), {
Copy link
Collaborator

Choose a reason for hiding this comment

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

eslint校验提示:'props' is assigned a value but never used.eslint

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已解决

Copy link
Collaborator

@sudoooooo sudoooooo left a comment

Choose a reason for hiding this comment

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

@alwayrun 补充了lint检测提示,请优化

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