Skip to content

set explicit limit when listing organizations#2005

Open
sakethsomaraju wants to merge 2 commits intomainfrom
fix/1974
Open

set explicit limit when listing organizations#2005
sakethsomaraju wants to merge 2 commits intomainfrom
fix/1974

Conversation

@sakethsomaraju
Copy link

Description

Describe the purpose of this pull request.

Set an explicit limit when listing organisations

🎟 Issue(s)

Related #1974

🧪 Functional Testing

./astro organization switch 
#   NAME                                   ID
1   Organization A                         org_a9f3k61tk008t01lpduvhq01
2   Organization B                         org_b1ojnif514gu01q1h6gi402
3   Organization C                         org_c7m6sqsr0dup01mziqj3dh
4   Organization D                         org_d74r2a000ia501jxb3zecy
5   Organization E                         org_e1deotszu0bx40txq4hs39
6   Organization F                         org_f9v2n8o204i401hvdms70b
7   Organization G                         org_glyencvlg051701g5nuy35
8   Organization H                         org_hhajccrv17v601kb14gt3m
9   Organization I                         org_if2bq65w2n6x01psvqqz5c
10  Organization J                         org_jlzvofruy0h8n01htzvl6n
11  Organization K                         org_kmf35hf6s2v0i01o4k9d3o
12  Organization L                         org_lm0gx1hiu008c01llx94j2k
13  Organization M                         org_mmbb539wy0nsc01lrvj1g0r
14  Organization N                         org_nmafgwb8c0olq01mgp8k9w2
15  Organization O                         org_olpsyubyx00aa01nk371zi
16  Organization P                         org_pclilt7rt8009l01i7wf7x
17  Organization Q                         org_qclqngdd1m018v01p0xkl8
18  Organization R                         org_rclksfpyij003x01nfaqz8
19  Organization S                         org_sclddgzqej00kv0tzf97r5
20  Organization T                         org_tcluu3c4uy018601orkdvl
21  Organization U                         org_ucknaqyipv05731evsry6c
>

List the functional testing steps to confirm this feature or fix.

Ran astro organization switch with a user assigned to more than 20 orgs.

📸 Screenshots

NA since public repo.

📋 Checklist

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before taking out of draft
  • Added/updated applicable tests
  • Tested against Astro-API (if necessary).
  • Tested against Houston-API and Astronomer (if necessary).
  • Communicated to/tagged owners of respective clients potentially impacted by these changes.
  • Updated any related documentation

@coveralls-official
Copy link

Pull Request Test Coverage Report for Build 241611e1-5533-458e-8343-a061bd4c9134

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 33.153%

Totals Coverage Status
Change from base Build 63595748-3be8-42e5-8f39-9a30d684e069: 0.003%
Covered Lines: 20868
Relevant Lines: 62944

💛 - Coveralls

@sakethsomaraju sakethsomaraju marked this pull request as ready for review February 16, 2026 03:28
@sakethsomaraju sakethsomaraju requested a review from a team as a code owner February 16, 2026 03:28
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.

1 participant

Comments