Move bootstrap-notify functionality to a service

ACRA / acralyzer

Open source backend for ACRA reports.

GNU General Public License v3.0

489 stars 90 forks source link

Closed halkeye closed 11 years ago

halkeye commented 11 years ago

Instead of littering the code with:

$('.top-right').notify({
    message: { text: '...' },
    type: 'info'
}).show();

I'm proposing moving the code to a service so we can just do:

bootstrapNotify.info("message");
bootStrapNotify.error("err msg");

halkeye commented 11 years ago

Aww. I can't add labels or assigns. I'll be working on this soon and provide a pull request.

halkeye commented 11 years ago

Actually I'm going to roll desktop notifications into this as well.

halkeye commented 11 years ago

this is actually #16 so closing this bug.