Skip to content
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

Handbook: Import the post_modified from last Github commit date #505

Open
renintw opened this issue Feb 19, 2024 · 0 comments
Open

Handbook: Import the post_modified from last Github commit date #505

renintw opened this issue Feb 19, 2024 · 0 comments

Comments

@renintw
Copy link
Contributor

renintw commented Feb 19, 2024

Per #456 the last modified date of github-imported handbooks can be incorrect.

While the above issue has been closed via a once-off fix, we should move forward with querying the last commit upon markdown import, such that the dates (which are now prominently displayed) are accurate.

To make it work with standalone environments, as well as production, we can use a GitHub App if available and fall back to unauthenticated API requests if not. That should likely be enough to mostly-work™ for local testing, while working perfectly in production.

Note: The markdown importer is not contained within this repo, but this is the primary place it's used and this bug is visible. The source is here: https://github.com/WordPress/wordpress.org/tree/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant