Closed
Bug 881595
Opened 8 years ago
Closed 8 years ago
Prevent ugly error with duplicate titles
Categories
(Webmaker Graveyard :: Thimble, defect)
Webmaker Graveyard
Thimble
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: brett, Assigned: michiel)
References
Details
Attachments
(1 file)
Currently if you try and create a new make with a title identical to a make in your dashboard, thimble throws a gnarly error that includes: "Sorry, an error occurred while trying to publish. Error: You already have a page named test at /var/www/thimble" Lets replace this with a more friendly dialogue that says "You already have a make with this name. Please choose another title using the <title> tag" and exclude the extraneous error messages.
agreed. There's also https://bugzilla.mozilla.org/show_bug.cgi?id=873499, which is the more general "clean up our errors", but doing them on a case by case basis lets us track better. Marking as blocking 881595, if you run across other errors that have bad error messages, too, please file those as well and assign to me.
Blocks: 873499
Status: NEW → ASSIGNED
Attachment #760933 -
Flags: review?(scott)
Attachment #760933 -
Flags: review?(scott) → review?(david.humphrey)
Updated•8 years ago
|
Attachment #760933 -
Flags: review?(david.humphrey) → review+
Comment 3•8 years ago
|
||
Commit pushed to master at https://github.com/mozilla/thimble.webmaker.org https://github.com/mozilla/thimble.webmaker.org/commit/cacb1bd645c4ea73478a296c4ddc10e0a847dd38 Merge pull request #109 from Pomax/bug881595 turned Error object into string error
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•