Skip to content

[REST] avatar_url examples are misleading #3259

Open
@jsoref

Description

@jsoref

Code of Conduct

  • I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?

https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28

What part(s) of the article would you like to see updated?

The example responses have things like:

  {
    "url": "https://api.github.com/repos/octocat/Hello-World/statuses/6dcb09b5b57875f334f61aebed695e2e4193db5e",
    "avatar_url": "https://github.com/images/error/hubot_happy.gif",
    "id": 1,
    "node_id": "MDY6U3RhdHVzMQ==",
    "state": "success",
    "description": "Build has completed successfully",
    "target_url": "https://ci.example.com/1000/output",
    "context": "continuous-integration/jenkins",
    "created_at": "2012-07-20T01:19:13Z",
    "updated_at": "2012-07-20T01:19:13Z",
    "creator": {
      "login": "octocat",
      "id": 1,
      "node_id": "MDQ6VXNlcjE=",
      "avatar_url": "https://github.com/images/error/octocat_happy.gif",

Note that both avatar_urls are 404:

This doesn't seem to match the real world, and since I'm looking for the word error, it's really confusing/misleading.

Checking https://api.github.com/users/octocat, it appears it's more likely that it should be something like:

{
  "login": "octocat",
  "id": 583231,
  "node_id": "MDQ6VXNlcjU4MzIzMQ==",
  "avatar_url": "https://avatars.githubusercontent.com/u/583231?v=4",

I can't tell if that's the avatar_url I should see in both places or if they're very different things.

Additional information

I'm trying to get information on error, failure, pending, or success, so I'm looking for those in the output of responses...

Note that the use of a lot of "id": 1, for things that might be different is very counter-productive.

It'd be really nice if the sample output covered each of the status states instead of just one.

Activity

codewithdev

codewithdev commented on Dec 20, 2023

@codewithdev

Hey team, happy to pick it up. 😄

jsoref

jsoref commented on Dec 20, 2023

@jsoref
Author

fwiw, REST content can't be fixed by externals

codewithdev

codewithdev commented on Dec 20, 2023

@codewithdev

@jsoref Did you ran your spell-checker for GitHub Docs as well? like others? 😁

jsoref

jsoref commented on Dec 20, 2023

@jsoref
Author

Yes. And most of my fixes are merged.

nguyenalex836

nguyenalex836 commented on Dec 20, 2023

@nguyenalex836

@jsoref Thank you for opening this issue! I'll get this triaged for review ✨

docs-bot

docs-bot commented on Dec 20, 2023

@docs-bot
Collaborator

Thank you for opening this issue! Changes to the REST API schema can be requested in github/rest-api-description. I will transfer your issue over to that open source repo.

transferred this issue fromgithub/docson Dec 20, 2023
SomeTroglodyte

SomeTroglodyte commented on Jan 8, 2024

@SomeTroglodyte
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @jsoref@codewithdev@becco@SomeTroglodyte@docs-bot

        Issue actions

          [REST] avatar_url examples are misleading · Issue #3259 · github/rest-api-description