Skip to content

fix: Default to owner username when no default org received #3910

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 10, 2025
Merged

fix: Default to owner username when no default org received #3910

merged 1 commit into from
Jul 10, 2025

Conversation

ElioDiNino
Copy link
Member

@ElioDiNino ElioDiNino commented Jul 10, 2025 •
edited
Loading

When no default org exists currently, users are shown an empty page. This change fixes that behaviour to show the user’s personal org instead.

@ElioDiNino ElioDiNino self-assigned this Jul 10, 2025
@ElioDiNino ElioDiNino requested a review from spalmurray July 10, 2025 17:00
@codecov Codecov
Copy link

codecov bot commented Jul 10, 2025 •
edited
Loading

Bundle Report

Changes will increase total bundle size by 60 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-production-system 6.23MB 19 bytes (0.0%) ⬆️
gazebo-production-esm 6.31MB 41 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-production-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 19 bytes 716.78kB 0.0%

Files in assets/index-legacy.*.js:

  • ./src/App.tsx → Total Size: 8.65kB
view changes for bundle: gazebo-production-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 41 bytes 639.33kB 0.01%

Files in assets/index.*.js:

  • ./src/App.tsx → Total Size: 8.5kB

@codecov-staging
Copy link

codecov-staging bot commented Jul 10, 2025 •
edited
Loading

Bundle Report

Changes will increase total bundle size by 60 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
gazebo-staging-system 6.23MB 19 bytes (0.0%) ⬆️
gazebo-staging-esm 6.31MB 41 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: gazebo-staging-system

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-legacy.*.js 19 bytes 716.78kB 0.0%

Files in assets/index-legacy.*.js:

  • ./src/App.tsx → Total Size: 8.65kB
view changes for bundle: gazebo-staging-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index.*.js 41 bytes 639.33kB 0.01%

Files in assets/index.*.js:

  • ./src/App.tsx → Total Size: 8.5kB

@codecov-qa
Copy link

codecov-qa bot commented Jul 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.66%. Comparing base (b5308dd) to head (1a62c11).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3910      +/-   ##
==========================================
+ Coverage   98.64%   98.66%   +0.01%     
==========================================
  Files         828      828              
  Lines       15083    15083              
  Branches     4319     4319              
==========================================
+ Hits        14879    14881       +2     
+ Misses        196      194       -2     
  Partials        8        8
Files with missing lines Coverage Δ
src/App.tsx 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.18% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.15% <ø> (+0.14%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5308dd...1a62c11. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codecov Codecov
Copy link

codecov bot commented Jul 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.66%. Comparing base (b5308dd) to head (1a62c11).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3910      +/-   ##
==========================================
+ Coverage   98.64%   98.66%   +0.01%     
==========================================
  Files         828      828              
  Lines       15083    15083              
  Branches     4311     4311              
==========================================
+ Hits        14879    14881       +2     
+ Misses        196      194       -2     
  Partials        8        8
Files with missing lines Coverage Δ
src/App.tsx 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.18% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.15% <ø> (+0.14%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5308dd...1a62c11. Read the comment docs.

@codecov-public-qa Codecov Public QA
Copy link

codecov-public-qa bot commented Jul 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.66%. Comparing base (b5308dd) to head (1a62c11).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3910      +/-   ##
==========================================
+ Coverage   98.64%   98.66%   +0.01%     
==========================================
  Files         828      828              
  Lines       15083    15083              
  Branches     4319     4319              
==========================================
+ Hits        14879    14881       +2     
+ Misses        196      194       -2     
  Partials        8        8
Files with missing lines Coverage Δ
src/App.tsx 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.18% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.15% <ø> (+0.14%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5308dd...1a62c11. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codecov-releaser
Copy link
Contributor

✅ Deploy preview for gazebo ready!

Previews expire after 1 month automatically.

Storybook

Commit Created Cloud Enterprise
1a62c11 Thu, 10 Jul 2025 17:05:11 GMT Cloud Enterprise

@codecov-notifications
Copy link

codecov-notifications bot commented Jul 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@            Coverage Diff             @@
##             main    #3910      +/-   ##
==========================================
+ Coverage   98.64%   98.66%   +0.01%     
==========================================
  Files         828      828              
  Lines       15083    15083              
  Branches     4311     4311              
==========================================
+ Hits        14879    14881       +2     
+ Misses        196      194       -2     
  Partials        8        8
Files with missing lines Coverage Δ
src/App.tsx 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Components Coverage Δ
Assets 100.00% <ø> (ø)
Layouts 99.71% <ø> (ø)
Pages 98.18% <ø> (ø)
Services 99.32% <ø> (ø)
Shared 99.12% <ø> (ø)
UI 99.15% <ø> (+0.14%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5308dd...1a62c11. Read the comment docs.

@ElioDiNino ElioDiNino enabled auto-merge July 10, 2025 17:06
ajay-sentry
ajay-sentry reviewed Jul 10, 2025
View reviewed changes
@ElioDiNino ElioDiNino added this pull request to the merge queue Jul 10, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jul 10, 2025
@ElioDiNino
fix: Default to owner username when no default org received (#3910)
74ea69d
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 10, 2025
@ElioDiNino ElioDiNino added this pull request to the merge queue Jul 10, 2025
Merged via the queue into main with commit e00cf1f Jul 10, 2025
64 checks passed
@ElioDiNino ElioDiNino deleted the ElioDiNino/default-org-fallback branch July 10, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajay-sentry ajay-sentry ajay-sentry approved these changes

@spalmurray spalmurray Awaiting requested review from spalmurray

Assignees

@ElioDiNino ElioDiNino

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ElioDiNino @codecov-releaser @ajay-sentry