Docs/feedback cleanup #21
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yoshizzle
requested changes
Dec 20, 2017
src/data/objects/image.yaml
Outdated
| @@ -39,7 +39,6 @@ schema: | |||
| is_public: | |||
| type: Boolean | |||
| value: false | |||
| filterable: true | |||
There was a problem hiding this comment.
is_public is filterable per https://bits.linode.com/LinodeAPI/apinext/blob/development/apinext/maps/image.py#L53
src/data/objects/profile.yaml
Outdated
| @@ -50,7 +50,7 @@ schema: | |||
| value: 2 | |||
| description: Total number of referrals attributed to user that have not yet converted to full accounts. | |||
| credit: | |||
| type: Integer | |||
| type: Float | |||
| value: 160 | |||
There was a problem hiding this comment.
if we're doing 40.0 as the example for longviewsubscription.yaml, shouldn't we also add the ".0" at the end of all of these other US dollar float value examples as well?
yoshizzle
approved these changes
Dec 20, 2017
jamesottinger
suggested changes
Dec 22, 2017
src/data/objects/image.yaml
Outdated
| @@ -39,7 +39,7 @@ schema: | |||
| is_public: | |||
| type: Boolean | |||
| value: false | |||
| filterable: true | |||
| fitlerable: true | |||
jamesottinger
approved these changes
Dec 22, 2017
alioso
pushed a commit
that referenced
this pull request
Aug 22, 2019
…-cli new: Added x-linode-cli-skip to actions the CLI can't support
LBGarber
pushed a commit
to LBGarber/linode-api-docs
that referenced
this pull request
Jun 15, 2022
Capitalized the L in Linode
LBGarber
pushed a commit
to LBGarber/linode-api-docs
that referenced
this pull request
Jun 15, 2022
* updating footer structure and content to match new marketing site * removing events display from homepage * removing all events and gql query for events * addressing PR feedback * refactoring menus * styling and markup for reg pages * more styling and api version * styling fix * add missing link * fix post rebase footer * configuring and light styling primary links * configuring and light styling utility links * styling sub menus * more styling sub menus * products menus and megamenu clicks * drop down behavior * adjust padding * add sub menus icons and descriptions * typography updates * submenus refactoring and architecture * wrap products menu and stles adjustments * add community menus * better accessibility * better accessibility * change names * change names * style updates * update styles and markup for consistency * coordinate styles with docs * post rebase fixes * fix rebase regressions * new logo and adjusting padding * adjusting styles to match * mobile menus switches * add mobile menus * mobile menu scripts and styling * adjustements bc of community * header spacing adjustemnt * styles update * mobile menus updates * mobile menus updates part 2 * add all producst links and more styling * add mobile sign up button * cherry picking footer changes from other branch * small chabge to logo wrapper * updating links, removing old links * post rebase fixes * fix dropdowns * update logo * update mobile styles * update mobile styles part 2 * correct height * regression fixes * regression fixes part 2 * adjustments to footer content & styles * adding mobile styles * slight modif to mid breakpoint primary nav * update logo * update home link * update logo again * removing tailwind refs and adding as regular css instead * whoops missing rem * fix @screen ref * style updates * style changes * updaing paths to us prod API * update footer and search links to point to prod * remove target blank from footer menus links
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addressing user feedback around docs inaccuracies