feat: Add dialog to execute Cloud Job with parameters

[mtrezza]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.

Summary by CodeRabbit

• New Features
  • Job execution now includes an optional parameter input dialog. Users can enter JSON parameters to configure job execution before confirming, providing greater flexibility in how jobs run.
  • JSON parameter validation is integrated to prevent execution with invalid parameters, while empty parameters are automatically accepted as valid defaults for streamlined operations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[parse-github-assistant]

🚀 Thanks for opening this pull request!

[coderabbitai]

Warning

Rate limit exceeded

@mtrezza has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 0 minutes and 2 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Two components are added to implement a dialog-based workflow for running jobs. A new RunJobDialog component provides a modal for entering JSON parameters, while RunNowButton is modified to open this dialog on click and execute the job with provided parameters on confirmation.

Changes

Cohort / File(s)	Summary
New RunJobDialog component src/dashboard/Data/Jobs/RunJobDialog.react.js	New class-based React component providing a modal dialog for running jobs with optional JSON parameters. Includes JSON validation, state management for params, and conditional confirm button enabling based on JSON validity.
RunNowButton dialog integration src/dashboard/Data/Jobs/RunNowButton.react.js	Integrates the new RunJobDialog with a showDialog state flag. Replaces immediate job execution on click with a two-step workflow: dialog open on click, then job execution on confirmation. Adds handleCancel and handleConfirm methods to manage dialog interactions.

Sequence Diagram(s)

sequenceDiagram
    actor User
    participant RunNowButton
    participant RunJobDialog
    participant JobSystem

    User->>RunNowButton: Click "Run Now" button
    RunNowButton->>RunJobDialog: Open dialog (showDialog=true)
    User->>RunJobDialog: Enter JSON parameters
    User->>RunJobDialog: Click Confirm
    RunJobDialog->>RunNowButton: Call onConfirm(jobWithParams)
    RunNowButton->>JobSystem: Execute job with params
    RunNowButton->>RunJobDialog: Close dialog (showDialog=false)
    JobSystem->>RunNowButton: Return execution status
    RunNowButton->>User: Display status ("Success!", "Failed.", etc.)

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~14 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is largely incomplete. While it includes the repository contribution notes, it's missing the core sections: the Issue link, Approach explanation, and Tasks checklist that are required by the template.	Complete the pull request description by filling in: (1) the 'Closes:' issue link, (2) an 'Approach' section explaining the dialog implementation and parameter handling, and (3) the 'Tasks' checklist indicating whether tests and documentation updates were completed.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a dialog for executing Cloud Jobs with parameters, which aligns with the implementation of RunJobDialog and integration with RunNowButton.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[parseplatformorg]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[parseplatformorg]

🎉 This change has been released in version 8.3.0-alpha.35