Skip to content

Conversation

@Kakulukian
Copy link
Member

@Kakulukian Kakulukian requested a review from julien-c November 8, 2023 13:09
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Nov 8, 2023

The documentation is not available anymore as the PR was closed or merged.

@Kakulukian Kakulukian changed the title Add notifice for "createdAt" key in API Add notice for "createdAt" key in API Nov 8, 2023
docs/hub/api.md Outdated

This is equivalent to `huggingface_hub.list_models()`.

The key `createdAt` in the response begins on `2022-03-02T23:29:04.000Z` as a result of a special migration.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO this should be mentioned just once, under a # Repo creation times heading, with a wording like:

For all repos-returning API calls, the createdAt attribute is the time when the repo was created.
Note that there is a special value of 2022-03-02T23:29:04.000Z for all repos that were created before we started storing the creation date.

cc'ing @Wauplin too

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah sure

Copy link
Member

@julien-c julien-c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but wdyt @Wauplin?

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Will integrate it to huggingface_hub as well 🔥

@Kakulukian Kakulukian merged commit 22e4120 into main Nov 9, 2023
@Kakulukian Kakulukian deleted the api-created-at branch November 9, 2023 15:59
@julien-c
Copy link
Member

julien-c commented Nov 9, 2023

looks fine in prod IMO, thanks:

image

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.

5 participants