Skip to content

0.2.1: da, li & ff for SNAP, option to disable automatic insertion - #2

Open
petermolnar wants to merge 1 commit into
jihaisse:masterfrom
petermolnar:master
Open

0.2.1: da, li & ff for SNAP, option to disable automatic insertion#2
petermolnar wants to merge 1 commit into
jihaisse:masterfrom
petermolnar:master

Conversation

@petermolnar

Copy link
Copy Markdown
Contributor

Question: should I create an admin options page for the automatic insertion?
I have an abstract class to take care most of the things, like saving options, w/ multisite support, I could move the plugin to those bases. ( See: https://github.com/petermolnar/wp-common )

@jihaisse

jihaisse commented Aug 1, 2014

Copy link
Copy Markdown
Owner

I don't know, is there is a need for an admin page ?
As the plugin is automatic, I don't see why it will need a admin options page. No ?

@petermolnar

Copy link
Copy Markdown
Contributor Author

The admin page would be there to apply it automatically or not, to maybe exclude services from the list, and so on.

@jihaisse

jihaisse commented Aug 1, 2014

Copy link
Copy Markdown
Owner

oh, ok.
Why not, sounds good to me.
Could you do that ? I can't merge because of conflicts, coul'd you make another pull request with your feature ?

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