Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Add a link to the account page. #308

Merged
merged 1 commit into from
Feb 11, 2021
Merged

Add a link to the account page. #308

merged 1 commit into from
Feb 11, 2021

Conversation

JulienPalard
Copy link
Contributor

Which is not obvious, I searched in admin.gandi.net for a bunch of
minutes before finding account.gandi.net.

@coveralls
Copy link

coveralls commented Feb 4, 2021

Coverage Status

Coverage remained the same at 90.743% when pulling 4f3caf0 on JulienPalard:mdk/link-to-account into ec0f3bb on Gandi:master.

@JulienPalard
Copy link
Contributor Author

Damned, it's more complicated than that. The account.gandi.net interface tells:

Votre nouvelle clef d'API est :

without telling if it's the xmlrpc api or apirest key.

Trying to set it as apirest key and running gandi domain list I get:

No configuration found, please use 'gandi setup' command

Trying to set it as api key and running gandi domain list I get:

Invalid API key, please use 'gandi setup' command.

so there's something that gandi setup should tell to clarify that, my PR is not enough. Do anyone know?

@akaihola
Copy link

akaihola commented Feb 5, 2021

@JulienPalard, see also #305.

Which is not obvious, I searched in admin.gandi.net for a bunch of
minutes before finding account.gandi.net.

Also v4/v5 being XMLRPC/REST split is not obvious.
@JulienPalard
Copy link
Contributor Author

It should be better now, I was able to query using my own instructions, but as I don't have a v4 key I only have access to:

 $ gandi dns create
 $ gandi dns delete
 $ gandi dns domain.list
 $ gandi dns list
 $ gandi dns keys create
 $ gandi dns keys delete
 $ gandi dns keys info
 $ gandi dns keys list
 $ gandi dns keys recover

@yanndinendal
Copy link
Member

Thanks :)

@yanndinendal yanndinendal merged commit bfe2e6f into Gandi:master Feb 11, 2021
@JulienPalard JulienPalard deleted the mdk/link-to-account branch February 11, 2021 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants