mr-fan Posted June 5, 2015 Share Posted June 5, 2015 (edited) So here i'm starting. This is more a little snippet than a whole module. On frontend before the </body> tag the script from http://browser-update.org/ is working - nothing fancy here. Download Github: https://github.com/mr-fan/MarkupBrowserUpdate my todo for this is: make it a clean module make use of the settings from browser-update.org so you could define these in PW backend get the backling for PW best regards mr-fan Edited June 5, 2015 by mr-fan 6 Link to comment Share on other sites More sharing options...
mr-fan Posted June 5, 2015 Author Share Posted June 5, 2015 So first question from a module newbee... 1. on installation a message is shown "session module unknown" so where is the issue here - naming? Link to comment Share on other sites More sharing options...
Soma Posted June 5, 2015 Share Posted June 5, 2015 Just an note: It's adding the script AFTER the </body> 1 Link to comment Share on other sites More sharing options...
mr-fan Posted June 5, 2015 Author Share Posted June 5, 2015 Since i was always on the road this week......this i just glued fast together....refactor the whole thing this weekend! Thanks for your notice! Link to comment Share on other sites More sharing options...
mr-fan Posted June 5, 2015 Author Share Posted June 5, 2015 Here we are with the first "real" version. First post edited. Name was changed in the right manner i think since it changes Markup it is a MarkupModule so no name confrontation with the HelloWorld modules on this front. @MOD/ADMIN please change the name of this topic to Module: MarkupBrowserUpdate ..cleared thank you Please test if you like the module. I'll try to get a backlink from browser-update.org for PW. regards mr-fan Link to comment Share on other sites More sharing options...
bernhard Posted June 9, 2015 Share Posted June 9, 2015 congratulations for your first real module - mine is still waiting one thing to this line: https://github.com/mr-fan/MarkupBrowserUpdate/blob/master/MarkupBrowserUpdate.module#L132 https://processwire.com/talk/topic/10106-api-problems-with-implementing-search/ 1 Link to comment Share on other sites More sharing options...
LostKobrakai Posted June 9, 2015 Share Posted June 9, 2015 @BernhardB In this case checking for the template should be enough as everything you navigate to in the backend is most likely a process module with the admin template (I'm not aware of any exceptions). It's only if you call pages by the api where you need to check by has_parent. E.g. when you edit an user the users page is of the template user, but the rendered page is actually an admin template with the module ProcessUser running. So you're not accessing other templates directly, but through process modules. 2 Link to comment Share on other sites More sharing options...
bernhard Posted June 9, 2015 Share Posted June 9, 2015 thanks for clarifying Link to comment Share on other sites More sharing options...
mr-fan Posted June 9, 2015 Author Share Posted June 9, 2015 congratulations for your first real module - mine is still waiting one thing to this line: https://github.com/mr-fan/MarkupBrowserUpdate/blob/master/MarkupBrowserUpdate.module#L132 https://processwire.com/talk/topic/10106-api-problems-with-implementing-search/ i'd take this line from the hello world module.....so i think it will do the job. ....i wouldn't name this a "real" module...one simple hook and some configurationfields but with PW fortunately it is so simple... But it is a good exercise to start with easy tasks....and if i've luck we get a backlink from browser-update.org for PW! But it is a good exercise to start with easy tasks....and if i've luck we get a backlink from browser-update.org for PW! just looked into github....and found https://github.com/browser-update/browser-update/commit/9a781c48f7931d473be3bfaab4a61d12c9288da3 2 Link to comment Share on other sites More sharing options...
mr-fan Posted July 22, 2015 Author Share Posted July 22, 2015 Just get that the missing backlink was there for processwire....now on http://browser-update.org/ homepage mission completed 1 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now