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:cronjob detail page #3914

Merged
merged 6 commits into from Sep 14, 2023
Merged

feat:cronjob detail page #3914

merged 6 commits into from Sep 14, 2023

Conversation

zjy365
Copy link
Contributor

@zjy365 zjy365 commented Sep 13, 2023

🤖 Generated by Copilot at 67c56f2

Summary

🕒🚀🐛

This pull request adds and updates several files related to the cronjob feature in the frontend. It improves the translation, style, and layout of the cronjob interface, adds new API functions and components to support different types of cronjobs and display more information about them, and fixes some bugs and typos in the code. It also adds two new dependencies: cron-parser and cronstrue.

We are the masters of the cronjob
We parse and display the schedule of doom
We fetch and show the logs and events
We delete the jobs with a single swoop

Walkthrough

  • Add and modify dependencies for cronjob feature in package.json (link)
  • Modify and add English and Chinese translations for cronjob labels and messages in common.json files (link, link, link, link, link, link, link, link)
  • Rename and add API functions to interact with cronjobs, jobs, pods, launchpad apps, and service accounts from the backend in api folder (link, link, link, link, link, link, link, link, link, link, link)
  • Rename and modify constants and types for cronjob feature in job.ts (link, link, link)
  • Modify the style and layout of the slider and table components for cronjob list and form in components folder (link, link, link, link)
  • Add and modify components to display cronjob details, history, events, and logs in detail folder (link, link, link, link, link, link, link, link)

@pull-request-size
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Sep 13, 2023

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 930
✅ Successful 390
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 539
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Patch has no changes to coverable lines.

📢 Thoughts on this report? Let us know!.

@zjy365 zjy365 force-pushed the cronjob branch 2 times, most recently from 8a5e06d to b3df4dd Compare September 13, 2023 08:49
Signed-off-by: jingyang <3161362058@qq.com>
@@ -71,11 +71,12 @@ const EditApp = ({ jobName, tabType }: { jobName?: string; tabType?: 'form' | 'y
debounce((data: CronJobEditType) => {
try {
console.log(data, 'data---');
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove

@c121914yu c121914yu merged commit b837bd3 into labring:main Sep 14, 2023
21 of 22 checks passed
bxy4543 pushed a commit to bxy4543/sealos that referenced this pull request Oct 13, 2023
* feat:cronjob detail page

Signed-off-by: jingyang <3161362058@qq.com>

* fix

* fix

* sort job

* pod log

* code

---------

Signed-off-by: jingyang <3161362058@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants