Skip to content

fix(auth): surface requestJoin failures in the org-setup alert (#4389) - #4397

Merged
PierreBrisorgueil merged 1 commit into
masterfrom
fix/requestjoin-error-surface
Jul 2, 2026
Merged

PierreBrisorgueil merged 1 commit into
masterfrom
fix/requestjoin-error-surface

Conversation

@PierreBrisorgueil

Copy link
Copy Markdown
Collaborator

Problem

requestJoin() in organizationSetup.component.vue only logged on failure — never set this.error, so the existing inline v-alert never rendered when a join request failed (network error, duplicate, permission). Sibling submit() handles it correctly.

Fix

Mirror submit(): this.error = null at the top of requestJoin() (clears a stale create-org error) and, in the catch, this.error = err?.response?.data?.message || err?.message || 'Could not send join request. Please try again.'. Dropped the console.error to match submit(). No template change.

Test

The component test mock exposed only createOrganization (so any requestJoin test threw before the catch). Added createJoinRequest to the mock + 7 tests: success, requestSent emit, failure→server message, fallback to err.message, generic fallback, stale-error clear, no-suggested-org guard.

Also (issue nit 2)

The never-committed, untracked src/lib/helpers/api.js + its test (which carried downstream references and were dead code) were deleted from the working tree — they were never on master, so nothing to change here; noted for the tracker.

Closes #4389

https://claude.ai/code/session_01V1mhCDsSUnn3WGtevd3YFX

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@PierreBrisorgueil, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 54 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: eb3c21b6-e785-4f60-bca3-1c7655f11a0e

📥 Commits

Reviewing files that changed from the base of the PR and between eb97682 and 0b8ef02.

📒 Files selected for processing (2)
  • src/modules/auth/components/organizationSetup.component.vue
  • src/modules/auth/tests/auth.organizationSetup.component.unit.tests.js
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/requestjoin-error-surface

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.

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.56%. Comparing base (4bc2515) to head (0b8ef02).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4397   +/-   ##
=======================================
  Coverage   99.56%   99.56%           
=======================================
  Files          35       35           
  Lines        1390     1390           
  Branches      435      435           
=======================================
  Hits         1384     1384           
  Misses          6        6           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@PierreBrisorgueil
PierreBrisorgueil merged commit f92003d into master Jul 2, 2026
7 checks passed
@PierreBrisorgueil
PierreBrisorgueil deleted the fix/requestjoin-error-surface branch July 2, 2026 14:55
PierreBrisorgueil pushed a commit that referenced this pull request Aug 1, 2026
# [2.3.0](v2.2.0...v2.3.0) (2026-08-01)

### Bug Fixes

* **auth,organizations:** guard soft-refresh navigation and the signout race ([#4464](#4464)) ([6a2546e](6a2546e)), closes [#4451](#4451) [#4459](#4459)
* **auth,organizations:** soft-refresh (token) at onboarding sites to prevent silent sign-out ([#4431](#4431)) ([5d88766](5d88766))
* **auth,organizations:** surface failures in org-create and email-verification flows ([#4451](#4451)) ([b161df5](b161df5)), closes [#4431](#4431) [#4447](#4447)
* **auth:** surface requestJoin failures in the org-setup alert ([#4389](#4389)) ([#4397](#4397)) ([f92003d](f92003d))
* **auth:** token() soft-refresh also carries pendingRequests ([#4436](#4436)) ([0b40e8c](0b40e8c))
* **billing:** config-source the upgrade-prompt pricing copy ([#4465](#4465)) ([9cb370d](9cb370d)), closes [#4460](#4460)
* **billing:** meter-mode upgrade prompt copy + single-entry route ([#4410](#4410)) ([df396cf](df396cf)), closes [pricing#units](https://github.com/pricing/issues/units) [pricing#units](https://github.com/pricing/issues/units)
* **billing:** migrate default packs to the V4 pricing-card schema ([#4463](#4463)) ([84789b8](84789b8)), closes [pricing#units](https://github.com/pricing/issues/units) [#4458](#4458)
* **core,organizations:** config-gate org-required header + unify first-org redirect ([#4423](#4423)) ([7adb298](7adb298)), closes [#4421](#4421) [#4422](#4422)
* **core:** wire the computed theme into Vuetify so auto dark-mode works ([#4466](#4466)) ([ed35252](ed35252)), closes [#4462](#4462)
* **prerender:** strip local server origin from captured HTML + fail-hard leak assert ([#4502](#4502)) ([#4503](#4503)) ([dbcf8d7](dbcf8d7))
* **skills:** flip PR to ready before waiting on CodeRabbit to avoid a draft deadlock ([#4473](#4473)) ([a733998](a733998)), closes [#4450](#4450)

### Features

* **billing:** pricing feature rows wrap + explicit ✗ not-included marker ([#4390](#4390)) ([4bc2515](4bc2515))
* **invitations:** show invite form when user-facing invitations enabled with open signup ([#4501](#4501)) ([daa6688](daa6688)), closes [#4500](#4500)
* **skills:** /feature non-interactive invocation — structured SKIP/STOP instead of prompts ([#4455](#4455)) ([197c5f3](197c5f3)), closes [#N](https://github.com/pierreb-devkit/Vue/issues/N) [#failure](https://github.com/pierreb-devkit/Vue/issues/failure) [#4454](#4454)
* **ui:** config-overridable loader, plan-badge set, and copy strings ([#4475](#4475)) ([688911a](688911a)), closes [#4474](#4474)
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.

🐛 audit nits — requestJoin silent failure + uncommitted api.js helper

1 participant