Skip to content

Comments

Added a check to see if the public date is included in the shipper so we don't override it#437

Open
cmccully wants to merge 2 commits intomainfrom
public-from-shipper
Open

Added a check to see if the public date is included in the shipper so we don't override it#437
cmccully wants to merge 2 commits intomainfrom
public-from-shipper

Conversation

@cmccully
Copy link
Collaborator

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds functionality to set the public date from external metadata in the shipper message, preventing it from being overridden during processing. This is part of version 1.29.0.

Changes:

  • Added logic to check for and set public_date from file_info message during frame opening
  • Bumped version from 1.28.0 to 1.29.0
  • Updated changelog with release notes

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
pyproject.toml Version bump to 1.29.0
banzai/lco.py Added public_date initialization from file_info and minor formatting improvement
CHANGES.md Added release notes for version 1.29.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cmccully
Copy link
Collaborator Author

This closes #437

Copy link

@markBowman markBowman left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you

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.

2 participants