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

Substitute Freitagsfoo template if Event template isn't present #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

YtvwlD
Copy link
Member

@YtvwlD YtvwlD commented May 29, 2020

If we're just going to use {{Freitagsfoo | Date=1970-01-01 | Host=FIXME }} on Freitagsfoo pages, there won't be an Event template on that page.
Simply subsituting the template works.

(I think we should probably move the Event template to the general Freitagsfoo page, but that's a different story.)

@YtvwlD YtvwlD requested a review from derf May 29, 2020 00:28
@YtvwlD
Copy link
Member Author

YtvwlD commented May 29, 2020

Hold on, I'm not quite sure whether modifying @lines does actually really work that way or if it's just luck.

And it will severely decrease the performance – which currently isn't great.

I think we should probably switch to an SMW query. (That's just one round-trip then.)

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.

1 participant