Closed Bug 1161598 Opened 10 years ago Closed 10 years ago

Calendar's app module shouldn't expose things it only needs internally

Categories

(Firefox OS Graveyard :: Gaia::Calendar, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gaye, Assigned: gaye)

References

Details

Attachments

(3 files)

No description provided.
Assignee: nobody → gaye
Attachment #8601569 - Flags: review?(mmedeiros)
Comment on attachment 8601569 [details] [review] [gaia] gaye:move-more-app-stuff-out-of-app.js > mozilla-b2g:master r+ with minor comments on github. really good work. way easier to understand and moving codebase into the right direction.
Attachment #8601569 - Flags: review?(mmedeiros) → review+
Keywords: checkin-needed
Keywords: checkin-needed
http://docs.taskcluster.net/tools/task-graph-inspector/#P153gpPdTe6Lwd63t41r5Q The pull request failed to pass integration tests. It could not be landed, please try again.
Keywords: checkin-needed
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment on attachment 8603630 [details] [review] [gaia] gaye:bug-1161598-followup > mozilla-b2g:master r=self small followup
Attachment #8603630 - Flags: review+
Keywords: checkin-needed
Keywords: checkin-needed
http://docs.taskcluster.net/tools/task-graph-inspector/#wgHT2XbFRjGzzCn-jJBwOw The pull request failed to pass integration tests. It could not be landed, please try again.
Keywords: checkin-needed
Keywords: checkin-needed
Attachment #8603664 - Flags: review+
Keywords: checkin-needed
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: