Skip to content
This repository has been archived by the owner on Dec 16, 2020. It is now read-only.

Use API functions to load messages, use local storage instead of cookie whenever possible #73

Merged
merged 2 commits into from
Aug 29, 2016

Conversation

manicki
Copy link
Member

@manicki manicki commented Aug 25, 2016

Okay, so I messed up order of merging @WMDE-Fisch's pull requests and #71 and #72 didn't make it to master.
Here they come again then!

mw.loader.load( "//de.wikipedia.org/w/index.php?title=User:Christoph Fischer (WMDE)/Gadgets/DeepCat.css&action=raw&ctype=text/css" , "text/css" );
$.when( mw.loader.using( [ 'mediawiki.api.messages', 'mediawiki.jqueryMsg' ] ), $.ready ).done( function() {
mw.loader.load( "//de.wikipedia.org/w/index.php?title=User:Christoph Fischer (WMDE)/gadget-test.js&action=raw&ctype=text/javascript" );
mw.loader.load( "//de.wikipedia.org/w/index.php?title=User:Christoph Fischer (WMDE)/gadget-test.css&action=raw&ctype=text/css" , "text/css" );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this change (contains "test") is unintended, can you please revert it in this pull request?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point! done (old URLs restored)

@WMDE-Fisch
Copy link
Contributor

Thanks for fixing the test.js stuff ^^'

@WMDE-Fisch WMDE-Fisch deleted the fisch-chain branch August 29, 2016 07:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants