Skip to content

Partner success svg updated#3419

Merged
steven-tey merged 1 commit intomainfrom
apply-success-svg-replacement
Feb 5, 2026
Merged

Partner success svg updated#3419
steven-tey merged 1 commit intomainfrom
apply-success-svg-replacement

Conversation

@marcusljf
Copy link
Collaborator

@marcusljf marcusljf commented Feb 4, 2026

Updated the svg to reflect the current layout.

  • Uses the program logo and name like the current instance
  • Example link and rewards show for /steven and link to dub. Then we don't have to blur anything since it doesn't make mention of the person applying.
CleanShot 2026-02-04 at 11 23 37@2x

Summary by CodeRabbit

  • Style
    • Removed border and rounded corners from the screenshot on the partner application success page for a cleaner, streamlined appearance.

Updated the svg to reflect the current layout.
- Uses the program logo and name like the current instance
- Example link and rewards show for /steven and link to dub. Then we don't have to blur anything since it doesn't make mention of the person applying.
@vercel
Copy link
Contributor

vercel bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dub Error Error Feb 4, 2026 7:29pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

A styling change removes border and rounded corners from the Screenshot component in the partner application success page, while preserving height, width, and mask gradient properties.

Changes

Cohort / File(s) Summary
UI Styling Updates
apps/web/app/(ee)/partners.dub.co/(apply)/[programSlug]/(default)/apply/success/page.tsx
Removed border and rounded corners styling from Screenshot component while maintaining dimensions and mask gradient effects.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • steven-tey

Poem

With borders removed and corners now plain,
The screenshot sheds its decorative frame,
Smooth and unbounded, the rabbit hops free,
A minimalist vision for all eyes to see. 🐰✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'Partner success svg updated' does not accurately reflect the main change—it was about removing border and rounding from a Screenshot component, not updating SVG. Update the title to reflect the actual code change, such as 'Remove border and rounding from success page screenshot' or 'Update success page screenshot styling'.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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 apply-success-svg-replacement

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

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

@steven-tey steven-tey merged commit fdb0a7e into main Feb 5, 2026
10 of 12 checks passed
@steven-tey steven-tey deleted the apply-success-svg-replacement branch February 5, 2026 05:55
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