Closed
Bug 1021200
Opened 11 years ago
Closed 11 years ago
Move over apps/files
Categories
(Marketplace Graveyard :: Code Quality, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2014-06-24
People
(Reporter: andy+bugzilla, Assigned: robhudson)
Details
(Whiteboard: [repoman][clean-up])
Move the models, the tests and anything else relevant over into mkt and delete the relevant apps directory.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → robhudson.mozbugs
Assignee | ||
Comment 1•11 years ago
|
||
https://github.com/mozilla/zamboni/commit/5969ca0
To verify ensure we can submit a new app and also that the diff/file viewer in reviewer tools has no regressions.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•11 years ago
|
Target Milestone: --- → 2014-06-24
Comment 2•11 years ago
|
||
I get the following error when accessing a review attachment {"detail":"Internal Server Error"}
Is this an issue regarding the file viewer?
Please see screencast http://screencast.com/t/r2EXyVJvJ
Status: RESOLVED → VERIFIED
Flags: needinfo?(amckay)
Updated•11 years ago
|
Status: VERIFIED → RESOLVED
Closed: 11 years ago → 11 years ago
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Iulian Timis from comment #2)
> I get the following error when accessing a review attachment
> {"detail":"Internal Server Error"}
> Is this an issue regarding the file viewer?
> Please see screencast http://screencast.com/t/r2EXyVJvJ
That looks like a separate issue with "mkt/comm/". Could you please open a new bug?
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(amckay)
Comment 4•11 years ago
|
||
I have added Bug 1027663 . Could you please add some clear STR on how to test diff/file viewer in reviewer tools? Thank you
Flags: needinfo?(robhudson.mozbugs)
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to Iulian Timis from comment #4)
> I have added Bug 1027663 . Could you please add some clear STR on how to
> test diff/file viewer in reviewer tools? Thank you
There's no STR b/c nothing was broken, we just refactored the underlying code. As long as there are no regressions we're good.
Flags: needinfo?(robhudson.mozbugs)
You need to log in
before you can comment on or make changes to this bug.
Description
•