Skip to content

Revert "Fix /domains/import: linode-cli now displays import as an act… #210

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.

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

Merged

Conversation

nmelehan
Copy link
Contributor

@nmelehan nmelehan commented Mar 3, 2020

…ion"

This reverts commit b2a63c3.

@nmelehan
Copy link
Contributor Author

nmelehan commented Mar 3, 2020

The commit that this reverts had successfully added the domains import action to the CLI. However, the developer site would not build, and would throw this error:

/Users/nmelehan/Desktop/linode/developers/src/components/5_templates/api.js
  330:13  error  Cannot query field "x_linode_cli_command" on type "PathsPost". Did you mean "x_linode_cli_action", "x_linode_cli_skip", "x_linode_grant", or "x_linode_charge"?  graphql/template-strings

The usage of x-linode-cli-command for this commit was in line with how it's placed in other endpoints, so it's not yet clear why the error appears. Going to file this issue as an issue with the developers site.

CC @Dorthu

@nmelehan nmelehan requested a review from leslitagordita March 3, 2020 23:33
@nmelehan nmelehan merged commit 6a8fc65 into linode:development Mar 3, 2020
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
* Changelog for Cloud manager 0.48.0 and Api 4.0.15 (linode#184)

* Embed ReDocas iframe (linode#187)

* Add test path for testing iframe access on developers.dev (linode#188)

* Reactive path to /api/v4 (linode#189)

* Point Create Account button at login.linode.com (linode#191)

* Point Create Account button at login.linode.com

* Go directly to signup path

* Release 1.0.30: Use iframe to host ReDoc docs (linode#192)

* Release 1.0.30 (linode#190)

* Changelog for Cloud manager 0.48.0 and Api 4.0.15 (linode#184)

* Embed ReDocas iframe (linode#187)

* Add test path for testing iframe access on developers.dev (linode#188)

* Reactive path to /api/v4 (linode#189)

* Update api.html

* Update api.html

* Update api.html

* Testing iframe + base + allow X-Frame (linode#198)

* Fix iframe links (linode#199)

* Testing iframe + base + allow X-Frame

* Try _blank

* Reset env to develop for testing; remove <base> (linode#200)

* Remove base element (linode#201)

* DLC-115 fix mobile menu on iFramed api/v4 page (linode#205)

* Add changelog for API 4.0.16 (linode#203)

* Still have to set this manually until dynamic version is merged (linode#207)

* Changelogs for Cloud Manager 0.49.0 and API 4.0.17 (linode#204)

* Changelogs for Cloud Manager 0.49.0 and API 4.0.17

* Copy fixes

* Copy fixes (linode#206)

* fix few issues with changelogs formatting

* Post-merge fixes

* Update changelog dates (linode#210)

* Update changelog dates (linode#211)

* Update changelog dates

* Display dates updated

* Update development (linode#214)



* Release-1.0.31 (linode#208)

* Changelogs for Cloud Manager 0.49.0 and API 4.0.17 (linode#204)

* Changelogs for Cloud Manager 0.49.0 and API 4.0.17

* Copy fixes

* Copy fixes (linode#206)

* fix few issues with changelogs formatting

* Post-merge fixes

* Update release dates

* Changlog date updates

* DLC-115 fix mobile menu on iFramed api/v4 page (linode#205)

* Manager changelog 0.50.0 (linode#215)

* Add manager 0.50.0 changelog entry

* Copy edit

* Added API Changelog for version 4.0.18

* remove highlighting
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
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