mptwaktusolat / app_waktu_solat_malaysia

Prayer times app for Malaysia. Accurate data from JAKIM. Install the app on your phones now.
https://play.google.com/store/apps/details?id=live.iqfareez.waktusolatmalaysia
GNU General Public License v3.0
44 stars 21 forks source link

[feedback] Azan subuh berkumandang dilain waktu #173

Closed fire-bot closed 1 year ago

fire-bot commented 1 year ago

Sent by Malaysia Prayer Time MPT. Created by fire.


---------- Forwarded message ---------
From: MPT Support [mptwaktusolat@gmail.com](mailto:mptwaktusolat@gmail.com)
Date: Thu, 9 Feb 2023, 19:52
Subject: MPT report received (MPT-128A9)
To:

MPT Support

Dear user, thank you for submitting your valuable feedback.

On: Thursday, 9 February 2023, 19:40 MYT

You said that: Azan tidak tepat dengan waktu azan dan azan subuh berkumandang dilain waktu.

User email: __

App version: 2.7.3 (119)

Thanks for reaching out! This automated message lets you know we've received your feedback. The developer will respond to your feedback soon. Hang tight!

In the meantime, visit our GitHub to see about currently open issues.

To learn more about features and troubleshooting tips for the app, visit our website at waktusolat.iqfareez.com

iqfareez commented 1 year ago

Device info:

image

Related to #133, #136, #131, #115?

iqfareez commented 1 year ago

Changing this scope of this issue. Sorry, I'm not notice of this issue earlier until I've experienced it.

So the issue is, when user set to long azan, the azan will play Subuh's azan (with assolatu khairun min annaum), for every prayer time (zuhr, asar, etc.).

Also reported though review: image

I'll try to fix this issue today and inshallah, try to issue an update later today. 💪

iqfareez commented 1 year ago

This issue was caused by this commit https://github.com/mptwaktusolat/app_waktu_solat_malaysia/commit/15c6908e6837789fb4a493cb0d8b3aa581219d5e#diff-201d9a5130b365544310ceb6343c82f16aabe2377aa8bc11440b1aea865e1dc8.

I mistakenly assign all prayer times to be fajr azans. Astaghfirullah.

https://github.com/mptwaktusolat/app_waktu_solat_malaysia/blob/d94314aba202e7d604f7194b348a2c8c5f213c47/lib/notificationUtil/notification_scheduler.dart#L181

iqfareez commented 1 year ago

The fixes are simple, just replace back theazan_hejaz2013_fajr to azan_kurdhi2010 for affected prayer times.

But, as quoted from https://pub.dev/packages/flutter_local_notifications#custom-notification-icons-and-sounds

⚠️ For Android 8.0+, sounds and vibrations are associated with notification channels and can only be configured when they are first created. Showing/scheduling a notification will create a channel with the specified id if it doesn't exist already. If another notification specifies the same channel id but tries to specify another sound or vibration pattern then nothing occurs.

Which means, if I understand correctly, we need to change the notification channel id. So we need to gives a new Id to the affected channels.