🌱 Add ID attributr to Create button on Target profile creation modal

[nachandr]

Resolves Issue # 2774

PR Title Prefix

Every PR Title should be prefixed with :text: to indicate its type.

PR Title emoji

Types recognized:

• Breaking change: ⚠️ (:warning:)
• Non-breaking feature: ✨ (:sparkles:)
• Patch fix: 🐛 (:bug:)
• Docs: 📖 (:book:)
• Infra/Tests/Other: 🌱 (:seedling:)
• Integration/E2E tests: 🧪 (:test_tube:)
• No release note: 👻 (:ghost:)

For example, a pull request containing breaking changes might look like
:warning: My pull request contains breaking changes.

Since GitHub supports emoji aliases (ie. :ghost:), there is no need to include
the emoji directly in the PR title -- please use the alias. It used to be
the case that projects using emojis for PR typing had to include the emoji
directly because GitHub didn't render the alias. Given that :warning: is
easy enough to read as text, easy to parse in release tooling, and rendered in
GitHub well, we prefer to standardize on the alias.

For more information, please see the Konveyor Versioning Doc.

[coderabbitai]

Walkthrough

Added an id attribute to the submit button in the TargetProfileForm modal component, enhancing DOM targeting capabilities for testing and selection purposes without modifying functionality.

Changes

Cohort / File(s)	Summary
TargetProfileForm button enhancement client/src/app/pages/archetypes/components/target-profile-form.tsx	Added id attribute to submit button for improved DOM targeting

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single-line attribute addition with no logic or behavioral changes
• No impact on component control flow or functionality

Poem

A tiny id upon a button so fine,
Now testers can find it along the DOM line! 🐰✨
No logic was changed, just easier to find,
A small gift for selectors, so thoughtfully designed!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description only contains the repository's PR title template/guidelines, not an actual description of the changes, objectives, or motivation for this specific pull request.	Provide a meaningful description of the changes made, why they were made, and any relevant context or testing performed for this PR.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title contains a typo ('attributr' instead of 'attribute') and uses a seedling emoji prefix appropriately for a minor change, but clearly describes the main change of adding an ID to the Create button.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

client/src/app/pages/archetypes/components/target-profile-form.tsx (1)

186-194: Use a more descriptive ID following the existing naming convention.

The ID "submit" is too generic and inconsistent with other IDs in this file (e.g., "target-profile-name", "target-profile-generators-selector"). A more specific ID reduces the risk of conflicts and improves maintainability.

Apply this diff:

         <Button
           type="button"
-          id="submit"
+          id="target-profile-submit-button"
           variant="primary"
           isDisabled={!isValid}
           onClick={handleSubmit(submitToOnSave)}

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f578706 and c944ab8.

📒 Files selected for processing (1)

• client/src/app/pages/archetypes/components/target-profile-form.tsx (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: unit-test
• GitHub Check: build-and-upload-for-global-ci