Skip to content

fix: mailer links regression - #5166

Merged
jgoux merged 2 commits into
supabase:developfrom
7ttp:fix/regression-another
May 5, 2026
Merged

fix: mailer links regression#5166
jgoux merged 2 commits into
supabase:developfrom
7ttp:fix/regression-another

Conversation

@7ttp

@7ttp7ttp commented May 4, 2026•
edited
Loading

Copy link
Copy Markdown
Member

@7ttp
7ttp requested a review from a team as a code ownerMay 4, 2026 23:56
@7ttp
7ttp marked this pull request as draft May 5, 2026 00:03
@coveralls

coveralls commented May 5, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 25352353494

Coverage decreased (-0.01%) to 63.726%

Details

  • Coverage decreased (-0.01%) from the base build.
  • Patch coverage: 5 of 5 lines across 1 file are fully covered (100%).
  • 5 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

5 previously-covered lines in 1 file lost coverage.

FileLines Losing CoverageCoverage
internal/utils/git.go557.14%

Coverage Stats

Coverage Status
Relevant Lines:15664
Covered Lines:9982
Line Coverage:63.73%
Coverage Strength:7.02 hits per line
💛 - Coveralls

@7ttp
7ttp marked this pull request as ready for review May 5, 2026 01:15
@7ttp
7ttp requested review from avallete and jgouxMay 5, 2026 01:16
@7ttp7ttp added the bug Something isn't working label May 5, 2026
@7ttp7ttp self-assigned this May 5, 2026
@7ttp7ttp changed the title fix: mailer linksfix: mailer links regressionMay 5, 2026
jgoux
jgoux approved these changes May 5, 2026
View reviewed changes

@jgouxjgoux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for this and sorry for breaking it! 😓

@jgoux
jgoux merged commit e4a7ace into supabase:developMay 5, 2026
15 checks passed
@7ttp7ttp mentioned this pull request May 5, 2026
Closed
@7ttp
7ttp deleted the fix/regression-another branch May 5, 2026 10:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jgouxjgouxjgoux approved these changes
@avalleteavalleteAwaiting requested review from avallete

Assignees

@7ttp7ttp

Labels

bugSomething isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Magic-link action_link drops /auth/v1 prefix after #5092 (admin.generateLink returns 404)

3 participants

@7ttp@coveralls@jgoux