Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding startup crash icon fixes bug 732067 #440

Merged
1 commit merged into from
Mar 21, 2012
Merged

adding startup crash icon fixes bug 732067 #440

1 commit merged into from
Mar 21, 2012

Conversation

ghost
Copy link

@ghost ghost commented Mar 15, 2012

No description provided.

@rhelmer
Copy link
Contributor

rhelmer commented Mar 20, 2012

r+ from me, works as advertised

public function isStartupCrasher($signature, $start_date, $end_date) {
$summary_model = new Signature_Summary_Model();
$uptime = $summary_model->getSummary('uptime', $signature, $start_date, $end_date);
foreach($uptime as $up) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way to use the existing Signature_Summary_Model object to reduce queries and get this information?

@brandonsavage
Copy link
Contributor

r+ from me.

ghost pushed a commit that referenced this pull request Mar 21, 2012
…crashes-732067

adding startup crash icon fixes bug 732067
@ghost ghost merged commit 26b486f into mozilla-services:master Mar 21, 2012
rhelmer pushed a commit to rhelmer/socorro that referenced this pull request May 11, 2013
Remove Rock Your Firefox.  Bug 792507 killed it
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants