Skip to content

Conversation

@zhoufeng1989
Copy link
Contributor

@zhoufeng1989 zhoufeng1989 commented Aug 14, 2018

Fix issue #2207 and issue #2273, files of different folders are exported into seperate directories.

@tatoosh
Copy link

tatoosh commented Aug 14, 2018

linked #2273

@kazup01 kazup01 added the awaiting review ❇️ Pull request is awaiting a review. label Aug 14, 2018
notes.forEach(note => {
const noteDir = path.join(exportDir, folderKeyToName[note.folder], `${note.title}.md`)
if (note.type === 'MARKDOWN_NOTE') {
t.true(fs.existsSync(noteDir))
Copy link
Member

Choose a reason for hiding this comment

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

Let's check note contents too. Not just check the existence of the files.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @Rokt33r , done it!

@Rokt33r Rokt33r self-requested a review August 20, 2018 03:58
@Rokt33r Rokt33r added awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. and removed awaiting review ❇️ Pull request is awaiting a review. labels Aug 20, 2018
@Rokt33r Rokt33r added next release (v0.11.10) and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Sep 15, 2018
@Rokt33r Rokt33r merged commit 46df5a8 into BoostIO:master Sep 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants