Skip to content

Events: If the directory for the ICS file don't exist, make it#947

Open
ghost wants to merge 10 commits intomasterfrom
unknown repository
Open

Events: If the directory for the ICS file don't exist, make it#947
ghost wants to merge 10 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Oct 17, 2017

If you enable the "Events" plugin and make your website with the commande make html or make publish, you get an error message:

CRITICAL: FileNotFoundError: [Errno 2] Aucun fichier ou dossier de ce type: '…/output/calendar.ics'

This PR fix this problem.

Comment thread events/events.py Outdated

ical.add_component(ie)

# If the directory for the ICS file don't exist, make it
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested fixes for comment: # If the directory for the ICS file doesn't exist, create it

@justinmayer
Copy link
Copy Markdown
Member

Thanks for the fix, @Postroutine. I made a minor correction to the commented code. Once you've made the change, would you please rebase & squash your commits and then comment here when this PR is ready for another review?

@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 12, 2017

I correct the comment, do a fetch from master and an interactive rebase to push my 2 commits as last commits.

Hope I doing it correctly.

@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 22, 2017

@justinmayer: Any news?

I am wait the merging of this PR before rebase my other PR.

@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 16, 2018

@justinmayer: Any news?

@ghost ghost mentioned this pull request Mar 16, 2018
@justinmayer
Copy link
Copy Markdown
Member

Hi @Postroutine. As you can see in this pull request, there are a number of changes that ostensibly are unrelated to your intended changes. Do you see what I mean?

@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 17, 2018

Sincerely, no. Did I miss somthing?
Why the "Files changed" section see me so much changes when all only do 2 commit on this branch?
Did I do something wrong when I rebase & squash your commits ?

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.

7 participants