Repository navigation
repo-age - Use repository creation date - #10258
kumaranurag00748-byte wants to merge 3 commits into
Conversation
Use the GitHub REST API created_at value for repository age while preserving the first-commit link and date in the tooltip.
This comment was marked as resolved.
This comment was marked as resolved.
repo-age - Use repository creation date (#10012)repo-age - Use repository creation date
|
@fregante, if it looks good merge this pr, it should close the issue i opened. |
|
The PR is currently closed, but the requested change has been verified on the real GitHub page: https://github.com/NextAlone/Nagram The sidebar displays https://api.github.com/repos/NextAlone/Nagram The PR description includes the verification screenshot. Could a maintainer please reopen the PR for review? |
|
It can reuse the same API call, it doesn't need another one. If you test it and it works, the correct implementation should look something like this: https://github.com/refined-github/refined-github/pull/10182/changes |
Read createdAt from the existing GraphQL requests instead of making an additional REST API call.
…10012) Keep the age based on the first commit and show the repository creation date only when the dates differ substantially.
|
Would be good to get a screenshot of this new icon with the tooltip |
|
I added the required screenshot of tooltip in the PR description and changed the same accordingly. |
Closes #10012
What changed
Keep the repository age based on the first commit date, while comparing it with GitHub’s repository creation date from the existing GraphQL API request.
When the dates differ substantially, display a question-mark warning icon. Hovering the warning shows both the first commit date and repository creation date in the tooltip. The existing first-commit lookup and commits-page link are preserved.
Test URLs
Validation
npm testnpm run lintnpm run format:checknpm run buildAll checks passed.
Screenshot