extensions icon indicating copy to clipboard operation
extensions copied to clipboard

Markdown Blog Manager 1.0

Open pyronaur opened this issue 3 years ago • 8 comments

Description

Adding a Raycast extension to manage markdown file based blogs.

Checklist

pyronaur avatar Oct 16 '22 19:10 pyronaur

Congratulation on your new Raycast extension! :rocket:

We will review it shortly. Once the PR is approved and merged, the extension will be available on the Store.

raycastbot avatar Oct 17 '22 19:10 raycastbot

Whoa, thanks for reviewing, I've been using the extension for some time now and never ran into any of these issues. Glad to have someone else take a peek at it before it goes live 🙂

I'll have a look and get back to you with updates as soon as I can 👍

pyronaur avatar Oct 25 '22 09:10 pyronaur

Awesome 📿

It happens to me all the time when developing, sometimes the best thing to do is to uninstall and start fresh since we always code for happy flows 😊

I'm looking forward to testing this extension again 🔥

pernielsentikaer avatar Oct 25 '22 12:10 pernielsentikaer

You should consider using placeholders instead since it looks like this every time I enter the command

Done, I've switched the title to placeholder now

Consider using useForm instead, for better handling of the forms (and validation) - it's also possible to make a new post without a title.

The documentation seemed a bit incomplete at first, which is why I didn't pay much attention to useForm at first. Switched to useForm now and......its soooo much easier to handle forms that way 🤯

There is no folder path for public in the package.json file, shouldn't there be that?

Variable names throughout iterations got funky. Renamed, should be consistent now.

How does Category work, couldn't figure that out in the README

I've renamed "Category" to "Subdirectory". I use sub-directories as categories, but not everyone might want to do that. They are mentioned in the readme, so hopefully renaming made that clearer, but if it doesn't - I'll welcome any feedback that you may have on how to clarify further.

There is something funny going on when changing the title https://user-images.githubusercontent.com/38033107/197706487-587933eb-6970-42ec-bd87-507123ce5d87.mp4

That was really funky. Looks like the issue went away now that I'm using useForm.

pyronaur avatar Oct 26 '22 19:10 pyronaur

Do you think it could be an idea to auto-create the folders (draft/public) if they don't exist or at least handle the error telling the user to create the directories 😊

pernielsentikaer avatar Oct 27 '22 07:10 pernielsentikaer

I really wish there was a "Raycast way" of handling that workflow. A lot can go wrong, like if I attempt to create a directory that doesn't exist, but the specified path was wrong by one character at the start of the string, that would create a long invalid path of directories.

But for now - I think you're right, some sort of error should be surfaced and the setting unset if the specified path is invalid.

pyronaur avatar Oct 27 '22 07:10 pyronaur

Then maybe just check if the path exists and an error telling that the user has to create it instead?

pernielsentikaer avatar Oct 27 '22 08:10 pernielsentikaer

Sounds good 👍

pyronaur avatar Oct 27 '22 09:10 pyronaur

Hi @pernielsentikaer, I've wrapped both commands in a helper that checks if the paths are valid, and if they aren't - it'll show a somewhat user-friendly error message: Raycast-1

It's not looking as pretty as I'd want it to be, but I guess that's a limitation in Raycast for now. Or is there a better way?

pyronaur avatar Oct 29 '22 19:10 pyronaur

Hi @pyronaur 👋

It's much better now, could you look into this linter error, maybe one of the old files still exists?

pernielsentikaer avatar Nov 01 '22 13:11 pernielsentikaer

I think it didn't like that preferences exported both default and a named value. Should be ok now.

pyronaur avatar Nov 01 '22 15:11 pyronaur

Oooh. I'm using rsync without --delete to sync my own repository with raycast extensions. I need to update my workflow 😅

pyronaur avatar Nov 01 '22 15:11 pyronaur

Published to the Raycast Store: https://raycast.com/pyronaur/markdown-blog

raycastbot avatar Nov 01 '22 20:11 raycastbot

🎉🎉🎉 Thank you @pernielsentikaer for patiently walking through the code review with me. Truly appreciate it 👍

pyronaur avatar Nov 01 '22 21:11 pyronaur