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(components): [date-picker] basic-date-table #10344

Merged
merged 1 commit into from
Sep 10, 2023

Conversation

jw-foss
Copy link
Member

@jw-foss jw-foss commented Nov 1, 2022

  • Extract logic out of basic-date-table file.

Please make sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow contributing guide English | (中文 | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer to relative issues for your PR.

* Extract logic out of basic-date-table file.
@jw-foss jw-foss requested a review from a team November 1, 2022 07:08
@github-actions
Copy link

github-actions bot commented Nov 1, 2022

@jw-foss
Copy link
Member Author

jw-foss commented Nov 1, 2022

Since I've moved the location of the code, it is suggested that we can utilize some online diff checker for the code I've moved to make it easier for review. Such as DiffChecker

@github-actions github-actions bot added the CommitMessage::Qualified Qualified commit message label Nov 1, 2022
@github-actions
Copy link

github-actions bot commented Nov 1, 2022

🧪 Playground Preview: https://element-plus.run/?pr=10344
Please comment the example via this playground if needed.

@tolking tolking merged commit 06e8582 into dev Sep 10, 2023
@tolking tolking deleted the refactor/extract-logic-out-of-date-picker branch September 10, 2023 09:50
@element-bot element-bot mentioned this pull request Sep 14, 2023
3 tasks
consultation-applio pushed a commit to consultation-applio/element-plus that referenced this pull request Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CommitMessage::Qualified Qualified commit message Refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants