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

prevent future posts from showing in pins #169

Merged
merged 4 commits into from Sep 10, 2019
Merged

prevent future posts from showing in pins #169

merged 4 commits into from Sep 10, 2019

Conversation

ghost
Copy link

@ghost ghost commented Aug 12, 2019

this changes GetPinnedPosts to accept an includeFutre bool, which
returns future dated pinned posts when true.


  • I have signed the CLA

closes #158

this changes GetPinnedPosts to accept an includeFutre bool, which
returns future dated pinned posts when true.
collections.go Outdated Show resolved Hide resolved
database.go Outdated Show resolved Hide resolved
@ghost
Copy link
Author

ghost commented Aug 12, 2019

concerns addressed

@thebaer thebaer added this to the 0.11 milestone Sep 4, 2019
With the var there now, this makes the code a bit more readable.
@thebaer
Copy link
Member

thebaer commented Sep 10, 2019

Looks great, thanks! I just made a small change, now using the new isOwner var in the if statements. Merging now.

@thebaer thebaer merged commit 9256293 into develop Sep 10, 2019
@thebaer thebaer deleted the future-pins branch September 10, 2019 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Pinned scheduled posts are visible to all users before publish date
1 participant