Skip to content

Header/footer per category (without async loading)#26

Open
sm8ps wants to merge 2 commits intoenterprisemediawiki:masterfrom
sm8ps:master
Open

Header/footer per category (without async loading)#26
sm8ps wants to merge 2 commits intoenterprisemediawiki:masterfrom
sm8ps:master

Conversation

@sm8ps
Copy link
Copy Markdown

@sm8ps sm8ps commented Jul 30, 2019

Hi there! It has been a full four years since my first pull request with the same topic. Something must have gone wrong then, probably my misunderstanding of Git. Anyway, in the course of finally upgrading my wiki I was astonished to find out that my code did not make it to your repo and I have spent some hours getting acquainted with the code again.

I disable async loading in case there is a per category header or footer as I do not see an obvious way for dynamicload.js to know which category to look for. Though, to be honest, I do not quite exactly understand how it works.

Looking forward to hearing from you. Cheers!
Stefan Mueller, Switzerland

@sm8ps
Copy link
Copy Markdown
Author

sm8ps commented Oct 16, 2019

Hi there! Has anybody taken notice of this pull request?

@sm8ps
Copy link
Copy Markdown
Author

sm8ps commented Jun 13, 2020

Hi @jamesmontalvo3! I noticed that you merged another pull request the other day. Do you have time to look at this one here?

@jamesmontalvo3
Copy link
Copy Markdown
Contributor

I’ll take a look on Monday.

@sm8ps
Copy link
Copy Markdown
Author

sm8ps commented Sep 12, 2020

Hi @jamesmontalvo3! I have been fiddling with my Mediawiki installation lately which reminded me of my pull request here. Will you have time to look at it?
Cheers!

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