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

Automatically generate slug if not provided #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

badrihippo
Copy link
Contributor

This is an issue with Medium drafts, where a slug is not set yet. mediumexporter tries to save it using an empty string (""), which results in a file named .md or the images folder being created in the main content directory rather than in a subfolder.

By automatically setting the slug, mediumexporter can ensure that each story gets saved in its own directory

This is an issue with Medium drafts, where a slug is not set yet.
mediumexporter tries to save it using an empty string (""), which
results in a file named `.md` or the `images` folder being created
in the main content directory rather than in a subfolder.

By automatically setting the slug, mediumexporter can ensure that
each story gets saved in its own  directory
@codecov-io
Copy link

Codecov Report

Merging #38 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #38      +/-   ##
=========================================
- Coverage    8.75%   8.69%   -0.07%     
=========================================
  Files           2       2              
  Lines         274     276       +2     
=========================================
  Hits           24      24              
- Misses        250     252       +2
Impacted Files Coverage Δ
lib/get-post.js 7.75% <0%> (-0.13%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bd2cb5c...010ea12. Read the comment docs.

@badrihippo
Copy link
Contributor Author

@xdamman one more bugfix! I've got a third one coming up—should I continue submitting separately, or would you prefer them all bundled up into a single pull request? I'll probably have a few more since I'm using mediumexporter for my own project so ironing out creases as I find them.

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.

2 participants