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

[docs] add last modification date to page Footer #29509

Merged
merged 8 commits into from
Jun 17, 2024

Conversation

Simek
Copy link
Collaborator

@Simek Simek commented Jun 6, 2024

Why

It might be valuable in some cases for docs users to know when the page they are viewing has been updated.

How

Add last modification date to page Footer based on the data from git about latest commit. The information is appended to the Markdown files before executing a deploy build at the cost of 30s-60s of CI run time. Locally, the data is not present, and we show a placeholder - this is the unsual pattern for docs frameworks including this functionality, i.e Docusaurus or GitBook.

Test Plan

The changes have been reviewed locally in development and static modes.

Preview

Screenshot 2024-06-06 at 16 36 59

@Simek Simek added the preview label Jun 6, 2024
@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Jun 6, 2024
@expo-bot
Copy link
Collaborator

expo-bot commented Jun 6, 2024

📘 Your docs preview website is ready!

@Simek Simek force-pushed the @simek/docs-show-page-latest-modification-date branch 2 times, most recently from 2122a52 to b8901b9 Compare June 7, 2024 09:02
@Simek Simek marked this pull request as ready for review June 7, 2024 09:19
@Simek Simek force-pushed the @simek/docs-show-page-latest-modification-date branch from 3026965 to f60f888 Compare June 8, 2024 21:36
Copy link
Member

@amandeepmittal amandeepmittal left a comment

Choose a reason for hiding this comment

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

Thank you! Looks amazing to me 🚀 Tested this locally by using `export-server.

Can I also ask you to add the modificationDate to https://docs.expo.dev/build-reference/infrastructure/ and remove the callout? 😄

@Simek Simek merged commit 39de0fd into main Jun 17, 2024
4 checks passed
@Simek Simek deleted the @simek/docs-show-page-latest-modification-date branch June 17, 2024 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants