Skip to content

Comments

Update README.rst - Draft PM#91

Draft
VladIftime wants to merge 7 commits intomainfrom
VladIftime-Kiflin
Draft

Update README.rst - Draft PM#91
VladIftime wants to merge 7 commits intomainfrom
VladIftime-Kiflin

Conversation

@VladIftime
Copy link
Contributor

@VladIftime VladIftime commented Dec 3, 2024

Draft

VladIftime and others added 7 commits November 29, 2024 14:52
Signed-off-by: VladIftime <49650168+VladIftime@users.noreply.github.com>
Signed-off-by: F.N. Claessen <felix@seita.nl>
… first, then pick out ourselves, locate our account ID, and then fetch our organisation account info (two API calls needed, I'm afraid)

Signed-off-by: F.N. Claessen <felix@seita.nl>
@nhoening
Copy link
Contributor

Is this still relevant, @VladIftime ?

I found get_account() is already on main, for instance

@nhoening
Copy link
Contributor

nhoening commented Feb 5, 2026

@Flix6x do you want to take a look a this and test if there is value here?

Both get_account() and test_get_account() (which take much % of this PR) are already on main (I didn't check if the implementation is equal however).

Feel free to close otherwise.

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.

3 participants