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

fix:初始化请求事件status更正 #8963

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

hsm-lv
Copy link
Collaborator

@hsm-lv hsm-lv commented Nov 30, 2023

What

🤖[deprecated] Generated by Copilot at 535b4fa

This pull request improves the error handling and feedback of the CRUD and service store components, and updates the documentation and examples of the CRUD and page components accordingly. It uses the updateMessage method to set the msg and msgType fields of the store with the error information, and modifies the msg argument of the toast action to include the responseStatus field.

🤖[deprecated] Generated by Copilot at 535b4fa

Sing, O Muse, of the code review that changed the CRUD component
How the wise developer added the responseStatus to the toast action
And how he handled errors with the updateMessage method
To show the user feedback, whether glorious or dreadful

Why

How

🤖[deprecated] Generated by Copilot at 535b4fa

  • Add fetchInited event to CRUD component and document its usage (link, link)
  • Call updateMessage method with error message and failure flag when API requests fail in CRUD and service stores (link, link, link, link, link)
  • Format matchSorter function signature for better readability in docs/zh-CN/components/crud.md (link)

@hsm-lv hsm-lv changed the title feat:初始化请求事件status更正 fix:初始化请求事件status更正 Nov 30, 2023
@hsm-lv hsm-lv requested a review from 2betop November 30, 2023 09:28
@github-actions github-actions bot added the feat label Nov 30, 2023
Copy link

👍 Thanks for this!
🏷 I have applied any labels matching special text in your issue.

Please review the labels and make any necessary changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant