Add a settings page to allow someone with manage_options permission to define static content URIs

[mrose17]

for non-programmers, requiring an edit to the plugin source to define the URIs and their contents is probably too much friction. for well-known URIs that have static content, the Setting page is sufficient...

also, tested the plugin on WP latest (4.6.1)

i hope you like this addition, thanks!

[pfefferle]

I like the idea, but I think supporting only three URIs is enough. We should support n-URIs and perhaps add a field for the mime type.

[mrose17]

@pfefferle - mission accomplished.

[mrose17]

@pfefferle - any thing else or is it ready for a merge? thanks!

[mrose17]

+1
Let me see what I can do tonight.

/mtr

On Sat, Oct 22, 2016 at 11:31 AM -0700, "Matthias Pfefferle" notifications@github.com wrote:

I like the idea, but I don't think supporting only three types is enough. We should support n-URIs and perhaps add a field for the mime type.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[pfefferle]

Thanks a lot for your work!

Here is some feedback:

• JSON is not working, because of escaping
• I would add the "Constants" as real constants of the "WellKnownPlugin" class
• do you think a custom post type will also work?

[mrose17]

thanks. i have addressed the first two issues. a custom post type would probably work, but then we are into a whole set of admin page edits.

my goal was to solve the problem for folks who want to populate their site with a single well-known URI without having to do any PHP programming. i think we've got that handled along with a lot of other things.

could we merge this PR and get it onto wordpress.org so i can point folks to it?

thanks!

[mrose17]

@pfefferle - sorry to nag, but "i'm on deadline"... can we get the PR merged and onto wordpress.org ? thanks!

[mrose17]

update: hi! i submitted the forked repo to wordpress.org, it's available as https://wordpress.org/plugins/well-known-uris/ ... if you merge this PR, i can retract that plugin and point people at this one.

sorry for the confusion!

best,

/mtr