Closed Bug 1073371 Opened 5 years ago Closed 5 years ago
[Voicemail] [Notification] 'SIM x' shown when receiving a vm notification and only 1 sim in device
### STR 1. Receive a voicemail notification ### Actual See Voicemail.png '(SIM 2)' shown before it. Not happening with missed call or message No difference if sim is in slot1 or 2 ### Version Gaia-Rev 86905e14c3ff06a0e6952ba635b6066ad2eea6b4 Gecko-Rev https://hg.mozilla.org/releases/mozilla-aurora/rev/572b666c54bd Build-ID 20140925160208 Version 34.0a2 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20140925.192608 FW-Date Thu Sep 25 19:26:18 EDT 2014 Bootloader L1TC10011800
Attachment #8495830 - Flags: ui-review?(echang) → ui-review+
Comment on attachment 8495829 [details] [review] Voicemail notification Sorry for the really late reply. I think :EragonJ should be the better person for this.
Attachment #8495829 - Flags: review?(timdream) → review?(ejchen)
Comment on attachment 8495829 [details] [review] Voicemail notification @Mancas, This patch is for System app, I think it would be better to ask for @Alive's review. (I checked your patch and please remember to fix unit tests)
Attachment #8495829 - Flags: review?(ejchen) → review?(alive)
Comment on attachment 8495829 [details] [review] Voicemail notification You need to fix voicemail_test.js as well.
Attachment #8495829 - Flags: review?(alive) → feedback+
Attachment #8498748 - Flags: review?(alive) → review+
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S6 (10oct)
Okay in v2.2 Gaia-Rev 9050edcda308b65d86577c8ed0eedc5c568d8e44 Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/0c8ae792f1c0 Build-ID 20141007160202 Version 35.0a1 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20141007.191609 FW-Date Tue Oct 7 19:16:20 EDT 2014 Bootloader L1TC00011840
You need to log in before you can comment on or make changes to this bug.