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

feat:初始化请求事件参数补充状态信息 #6832

Merged
merged 6 commits into from
May 9, 2023

Conversation

hsm-lv
Copy link
Collaborator

@hsm-lv hsm-lv commented May 9, 2023

What

🤖 Generated by Copilot at fc1a08c

This pull request adds new event parameters to several components that use api and schemaApi requests, such as service, crud, wizard, form, and page. These parameters include responseData, responseStatus, responseMsg, and error, and provide more information about the response data and status of the requests. This pull request also updates the documentation of these components to reflect the new event parameters, and improves the code quality and consistency of some components.

🤖 Generated by Copilot at fc1a08c

Sing, O Muse, of the mighty pull request
That added new parameters to the events
Of form, page, wizard, and service, the best
Of components, crafted by skillful developers.

Why

How

🤖 Generated by Copilot at fc1a08c

  • Add new event parameters to inited, fetchInited, fetchSchemaInited, finished, submitSucc, and submitFail events for various components, to provide more information about the response data and status of the api, initApi, and schemaApi requests (link, link, link, link, link, link, link, link, link, link, link)
  • Remove unused variables and redundant calls from the code of the crud and wizard components, to improve the code quality and avoid unnecessary destructuring and duplication (link, link)
  • Add a space between Alert and in the documentation of the showErrorMsg property for the service component, to improve the readability and formatting of the text (link)

@github-actions github-actions bot added the feat label May 9, 2023
@github-actions
Copy link

github-actions bot commented May 9, 2023

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

Please review the labels and make any necessary changes.

@hsm-lv hsm-lv requested review from 2betop and lurunze1226 May 9, 2023 05:23
@hsm-lv hsm-lv merged commit 50adcb2 into baidu:master May 9, 2023
3 checks passed
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.

None yet

3 participants