fix: adjust sponsor get function validation, set initial company id

[tomrndom]

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

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

Summary by CodeRabbit

• Bug Fixes
  • Improved data handling for sponsor company information to prevent errors when data is missing.
  • Enhanced performance by reducing unnecessary data fetches during sponsor request processing.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

Two files in the sponsor request management feature are modified: one ensures company ID defaults to 0 in initial form values when undefined, the other adds a conditional check to prevent fetching sponsor data when company ID is absent.

Changes

Cohort / File(s)	Summary
Sponsor Request Form/Popup src/pages/sponsors/sponsor-users-list-page/components/process-request-form.js, src/pages/sponsors/sponsor-users-list-page/components/process-request-popup.js	Default company.id to 0 when null/undefined in initial values; guard fetchSponsorByCompany execution to only run when request.company_id is truthy.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: add constants, change payload structure, new company on input #784: Modifies buildInitialValues in the same process-request-form.js file to normalize sponsor_type with a constant, sharing similar intent of initial value handling.

Suggested reviewers

• smarcet

Poem

🐰 A sponsor form hops with graceful care,
Defaulting IDs to zero, null beyond repair,
Conditional fetches skip the empty air,
No undefined shadows in the sponsor's lair! ✨

🚥 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 accurately describes the main changes: adjusting sponsor function validation and setting initial company id defaults.
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/sponsor-user-request-company-id

---

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

[smarcet]

LGTM