Skip to content

Comments

Add last_updated to metadata document#262

Merged
afragen merged 3 commits intofairpm:mainfrom
costdev:add_last_updated_to_metadata_document
Oct 27, 2025
Merged

Add last_updated to metadata document#262
afragen merged 3 commits intofairpm:mainfrom
costdev:add_last_updated_to_metadata_document

Conversation

@costdev
Copy link
Member

@costdev costdev commented Sep 23, 2025

Signed-off-by: costdev <79332690+costdev@users.noreply.github.com>
Signed-off-by: costdev <79332690+costdev@users.noreply.github.com>
@costdev costdev requested review from afragen and rmccue September 23, 2025 03:54
@afragen
Copy link
Contributor

afragen commented Sep 23, 2025

Needs fairpm/fair-beacon#54 and a spec update.

Copy link
Contributor

@afragen afragen left a comment

Choose a reason for hiding this comment

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

This will be correct when last_updated is added to mini-fair MetaDataDocument.

@afragen afragen changed the title Add last updated to metadata document Add last_updated to metadata document Sep 23, 2025
@afragen afragen marked this pull request as ready for review October 15, 2025 19:57
Signed-off-by: Andy Fragen <andy@thefragens.com>

Signed-off-by: Andy Fragen <andy@thefragens.com>
@Ipstenu Ipstenu added this to the 1.1 milestone Oct 27, 2025
@Ipstenu
Copy link
Contributor

Ipstenu commented Oct 27, 2025

@costdev Adding to 1.1 unless you think we should bump it to 1.2?

@costdev
Copy link
Member Author

costdev commented Oct 27, 2025

@Ipstenu I definitely want this "Last Updated 56 years ago" issue fixed for 1.1, and it's achievable.

While there's still discussion pending on the ideal location for the date, this PR and the MiniFAIR PR can resolve the issue without locking us into a specific location forever. The discussion, investigation, etc. on the ideal location should absolutely be resolved as soon as possible, and would be best done in the Protocol issue.

@afragen afragen merged commit 988fd63 into fairpm:main Oct 27, 2025
46 checks passed
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.

3 participants