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
Merged

Imroved WordPress page compiler plugin. #93

merged 1 commit into from Jul 13, 2015

Conversation

felixfontein
Copy link
Contributor

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;')
Copy link
Member

Choose a reason for hiding this comment

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

&gt;

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, fixed that.

@felixfontein felixfontein merged commit e65a725 into getnikola:master Jul 13, 2015
@felixfontein
Copy link
Contributor Author

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

@Kwpolska
Copy link
Member

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

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
Development

Successfully merging this pull request may close these issues.

None yet

2 participants