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

Add possibility to filter from a specific page. #20

Merged
merged 1 commit into from
Apr 5, 2020

Conversation

Lamecarlate
Copy link
Contributor

Hi,

my use case: I have blog items and portfolio items (and some other stuff), and I wanted to display archives only on (and for) blog items. If I used "@self" in blog.md, the archives would be correctly gathered on blog items list, but not on an blog item itself (which has no children).

So I used "@page: /blog" in my archives.yml, and changed the archives.php file to find all children of whatever "@page" could be.

Is this the correct way?

Hi,

my use case: I have blog items and portfolio items (and some other stuff), and I wanted to display archives only on (and for) blog items. If I used "@self" in blog.md, the archives would be correctly gathered on blog items list, but not on an blog item itself (which has no children).

So I used "@page: /blog" in my archives.yml, and changed the archives.php file to find all children of whatever "@page" could be.

Is this the correct way?
@ghost
Copy link

ghost commented May 2, 2019

Hi,

is there a reason not to merge this pull request? I have the same use case and would love to see this functionality added to the plugin :)

@rhukster
Copy link
Member

rhukster commented May 2, 2019

I'm just working my way through plugins at the moment and not gotten to reviewing archives plugin.

@rhukster rhukster self-assigned this Apr 3, 2020
@rhukster rhukster merged commit 5dae1e9 into getgrav:develop Apr 5, 2020
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