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

Emit an event when a post/page already exists, also. #1720

Merged
merged 1 commit into from May 14, 2015
Merged

Conversation

@punchagan
Copy link
Member

@punchagan punchagan commented May 13, 2015

Creation of new_post successfully emits an event, but no event is fired
new_post fails because there is an existing post. This event can be
used by plugins, for e.g. open the post in an editor or prompt to delete
the post, etc.

Creation of new_post successfully emits an event, but no event is fired
new_post fails because there is an existing post.  This event can be
used by plugins, for e.g. open the post in an editor or prompt to delete
the post, etc.
@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented May 14, 2015

Nobody will use this anyways, but it shouldn’t hurt. Merging.

Kwpolska added a commit that referenced this pull request May 14, 2015
Emit an event when a post/page already exists, also.
@Kwpolska Kwpolska merged commit 83c8254 into master May 14, 2015
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/travis-ci/push The Travis CI build failed
Details
continuous-integration/appveyor AppVeyor build succeeded
Details
@Kwpolska Kwpolska deleted the existing-post-event branch May 14, 2015
@Kwpolska Kwpolska added this to the v8.0.0 milestone May 14, 2015
@Kwpolska Kwpolska self-assigned this May 14, 2015
@ralsina

This comment has been minimized.

Copy link
Member

@ralsina ralsina commented on d33a023 May 14, 2015

These things should come with doc patches. Specifically:

https://github.com/getnikola/nikola/blob/master/docs/extending.txt#L391

This comment has been minimized.

Copy link
Member

@Kwpolska Kwpolska replied May 16, 2015

done 046002a

Kwpolska added a commit that referenced this pull request May 16, 2015
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

3 participants