Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Imroved WordPress page compiler plugin. #93

Merged
merged 1 commit into from Jul 13, 2015

Conversation

@felixfontein
Copy link
Contributor

felixfontein commented Jul 13, 2015

Added support for shortcode plugin data storage.
Adding additional data support to compile_to_string and compile_html.
Added basic gallery shortcode plugin.

(Uses extended metadata provided by getnikola/nikola#1888)

Adding additional data support to compile_to_string and compile_html.
Added basic gallery shortcode plugin.
def sanitize_html_text(text, sanitize_quotes=False):
text = text.replace('&', '&')
text = text.replace('<', '&lt;')
text = text.replace('>', '&gr;')

This comment has been minimized.

Copy link
@Kwpolska

This comment has been minimized.

Copy link
@felixfontein

felixfontein Jul 13, 2015

Author Contributor

Thanks, fixed that.

@felixfontein felixfontein merged commit e65a725 into getnikola:master Jul 13, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@felixfontein
Copy link
Contributor Author

felixfontein commented Jul 13, 2015

Whoops, accidentally pushed to the wrong repo. Gotta rename origin to something else :/

@Kwpolska
Copy link
Member

Kwpolska commented Jul 13, 2015

This is actually okay. You don’t really need to send PRs for this plugin, the website is updated automatically at midnight UTC and I just glanced through the code without really looking into it. As long as it works, just push to master and call it a day.

PS. please adjust the plugin version with changes.

@felixfontein
Copy link
Contributor Author

felixfontein commented Jul 13, 2015

Ah, good.
Added another small thing and bumped the version number (d8bdd56).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.