Skip to content

fix: add missing asterisks for rate fields on popups#782

Merged
smarcet merged 1 commit intomasterfrom
fix/add-asterisk-item-form
Feb 6, 2026
Merged

fix: add missing asterisks for rate fields on popups#782
smarcet merged 1 commit intomasterfrom
fix/add-asterisk-item-form

Conversation

@tomrndom
Copy link

@tomrndom tomrndom commented Feb 5, 2026

ref: https://app.clickup.com/t/86b84vefd

Signed-off-by: Tomás Castillo tcastilloboireau@gmail.com

Summary by CodeRabbit

  • Style
    • Added visual indicators (asterisks) to required form fields in inventory and sponsor pricing forms to improve clarity and user experience.

Signed-off-by: Tomás Castillo <tcastilloboireau@gmail.com>
@tomrndom tomrndom requested a review from smarcet February 5, 2026 21:20
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

Two form components updated to add required field indicators (asterisks) to rate field labels (early_bird_rate, standard_rate, onsite_rate). No functional logic or validation changes applied.

Changes

Cohort / File(s) Summary
Required Field Indicators
src/components/forms/inventory-item-form.js, src/pages/sponsors-global/form-templates/sponsor-inventory-popup.js
Added trailing asterisks to early_bird_rate, standard_rate, and onsite_rate field labels across both inventory forms to visually indicate required fields.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • smarcet

Poem

🐰 A sprinkle of stars in the form so bright,
Asterisks dancing, marking what's right,
Required fields now clearly defined,
Data collectors will surely find,
The fields they must fill without delay! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: add missing asterisks for rate fields on popups' directly and accurately summarizes the main change: adding asterisk indicators to rate field labels.
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/add-asterisk-item-form

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

Copy link

@smarcet smarcet left a comment

Choose a reason for hiding this comment

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

LGTM

@smarcet smarcet merged commit bfc71a4 into master Feb 6, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants