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.

Sign up for GitHub

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

Jump to bottom

fix: 修复pro/vue中部分问题，对接eggjs用户相关接口 #58

Merged
merged 21 commits into from
Jun 25, 2023
Merged

fix: 修复pro/vue中部分问题，对接eggjs用户相关接口 #58

merged 21 commits into from
Jun 25, 2023

Conversation

wjiangwang
Copy link
Contributor

@wjiangwang wjiangwang commented Jun 14, 2023 •
edited
Loading

PR

  • 修复MOCK数据情况下注册的用户无法登录问题

  • 修复i18n特殊字符导致邮箱登录页没有弹窗提示问题

  • 修复用户信息页控制台报错问题

  • 修复部分单词拼写错误

  • 删除部分 vue工程中不使用的代码及文件

  • pro套件vue模板与egg模板接口联调（登录、注册、查询修改用户信息、查询雇员列表）

  • 修改部分mock数据结构与egg保持一致

  • 后台未实现的接口继续使用mock数据（tiny init 选择对接后台。部分接口数据也是mock的）

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our Commit Message Guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@kagol
Copy link
Member

kagol commented Jun 20, 2023

@wjiangwang 需要在PR标题和描述里面说明清楚具体这个PR做了什么

@kagol kagol added the enhancement New feature or request label Jun 20, 2023
@wjiangwang wjiangwang changed the title Tiny pro vue fix: 修复pro/vue中部分问题，对接eggjs用户相关接口 Jun 21, 2023
h-ivy
h-ivy approved these changes Jun 25, 2023
View reviewed changes
Copy link
Contributor

@h-ivy h-ivy left a comment

Choose a reason for hiding this comment

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

ok

@h-ivy h-ivy merged commit 9e3685f into opentiny:dev Jun 25, 2023
@wjiangwang wjiangwang added the toolkit-pro toolkit-pro label Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hello-yezi hello-yezi hello-yezi left review comments

@aliceDxr aliceDxr aliceDxr left review comments

@kagol kagol kagol left review comments

@h-ivy h-ivy h-ivy approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request toolkit-pro toolkit-pro
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@wjiangwang @kagol @aliceDxr @hello-yezi @h-ivy