901447 - Missing Title Error should help the user understand that they need a title

Status: VERIFIED FIXED

(Webmaker Graveyard :: Thimble, defect)

Description

[Laura Hilliger [:epilepticrabbit]]

STR: go to thimble.webmaker.org
delete the text between the <title> and </<title>
hit publish
You will get this error:

Sorry, an error occurred while trying to publish. {"message":"Validation failed","name":"ValidationError","errors":{"title":{"message":"Validator \"required\" failed for path title","name":"ValidatorError","path":"title","type":"required"}},"__NR_CAUGHT":true}

We need to change that to let the user know that they do not have a title on their page. The above error confuses users, and because they can't publish, they get frustrated :)

Comment 1

[Pomax]

There should be code that injects the current date-time when a user publishes without a title. This suggests that codepath is not being hit, which is actually a bug. Thanks for the report!

Comment 2

[Scott [:thecount] Downe]

Pomax, can you take this?

I don't see myself getting to it anytime soon.

Comment 3

[Scott [:thecount] Downe]

Attachment: https://github.com/mozilla/thimble.webmaker.org/pull/217

An empty title should use the project id.

However, at some point the makeapi made title required and validate it so empty strings are not valid titles.

This patch now passes in the project id as the title to the makeapi too.

Comment 4

[Pomax]

Comment on attachment 806097 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/217

validation~~~~ *shakes fist in air*

Comment 5

[Scott [:thecount] Downe]

Staged: https://github.com/mozilla/thimble.webmaker.org/commit/23ec4a6c975dd2e7a72ebb07a4305032d7ed9121

Needs verification.