Skip to content

make slug field required - #50

Open
m-henderson wants to merge 1 commit into
madskristensen:masterfrom
m-henderson:slug-field-validation
Open

make slug field required#50
m-henderson wants to merge 1 commit into
madskristensen:masterfrom
m-henderson:slug-field-validation

Conversation

@m-henderson

Copy link
Copy Markdown

This will fix #49 but it does require the use to supply a slug.

@m-henderson

Copy link
Copy Markdown
Author

@madskristensen if you don't want the slug field to be required, we can automatically generate one.

@tbasallo

tbasallo commented Feb 2, 2018

Copy link
Copy Markdown

I've been meaning to auto-generate one based on the title in my fork. I think this would be ideal.

I wonder if there's a reason it wasn't considered.

@jimmylewis

Copy link
Copy Markdown
Contributor

I think #52 is the proper fix for this issue. Slug shouldn't be marked as required, as it is normally supposed to be generated from post.Title.

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.

Form Validation For Slug Input

3 participants