Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Conversation

@Shinsina
Copy link
Member

@Shinsina Shinsina commented Feb 15, 2022

Ref: #247

else: '$expires',
},
},
scheduled: '$startDate',
Copy link
Member

Choose a reason for hiding this comment

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

What's this intended to be used for?

Copy link
Member Author

Choose a reason for hiding this comment

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

Currently there isn't a field on sectionQuery if the schedule has a startDate set prior to the content published date (why this is a "thing" beats me). This adds one.

Screen Shot 2022-02-15 at 2 40 29 PM

Screen Shot 2022-02-15 at 2 40 24 PM

Copy link
Member

Choose a reason for hiding this comment

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

For what purpose? Section query intentionally uses published date to correct certain schedule start dates... in that example, that start date is actually wrong... you shouldn't be able to start content before it's actually published. Nothing in the interface prevents this, per se, hence why section query enforces this

Copy link
Member Author

@Shinsina Shinsina Feb 15, 2022

Choose a reason for hiding this comment

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

https://trello.com/c/XpVjx5vG/277-incorrect-date-being-displayed-on-content-list-items-for-non-primary-sites

I don't agree with the idea of putting dates prior to the publish date anywhere.

@zarathustra323
Copy link
Member

@Shinsina we're going to hold on these changes for now, pending some other options

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants