Closed Bug 1021200 Opened 11 years ago Closed 11 years ago

Move over apps/files

Categories

(Marketplace Graveyard :: Code Quality, defect, P2)

2014-Q2
x86
macOS
defect

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: nobody → robhudson.mozbugs
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
Target Milestone: --- → 2014-06-24
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)
Status: VERIFIED → RESOLVED
Closed: 11 years ago11 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?
Flags: needinfo?(amckay)
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)
(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)
Closing this bug then. Thank you!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.