Skip to content

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

Open
ghost wants to merge 10 commits into
masterfrom
unknown repository
Open

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

Conversation

@ghost

@ghost ghost commented Oct 17, 2017

Copy link
Copy Markdown

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

ghost commented Nov 12, 2017

Copy link
Copy Markdown
Author

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

ghost commented Nov 22, 2017

Copy link
Copy Markdown
Author

@justinmayer: Any news?

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

@ghost

ghost commented Mar 16, 2018

Copy link
Copy Markdown
Author

@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

ghost commented Mar 17, 2018

Copy link
Copy Markdown
Author

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