tirkarthi / NotificationDictionary

An android app to display meaning for selected word as notification
MIT License
149 stars 13 forks source link

Channel not created when the meaning is fetched prior to first app launch #37

Open VishnuSanal opened 1 year ago

VishnuSanal commented 1 year ago

No Channel found for pkg=com.xtreak.notificationdictionary, channelId=Dictionary, id=1, tag=null, opPkg=com.xtreak.notificationdictionary, callingUid=10599, userId=0, incomingUserId=0, notificationUid=10599, notification=Notification(channel=Dictionary shortcut=null contentView=null vibrate=null sound=null defaults=0x0 flags=0x10 color=0x00000000 vis=PRIVATE)

I think this is because of the creation of notification channel from MainActicity & not from ProcessTextActivity where the notification gets created.

tirkarthi commented 1 year ago

MainActivity downloads the dictionary and setup of the app is done. So checking for meaning without opening the app atleast once might be causing this.

VishnuSanal commented 1 year ago

MainActivity downloads the dictionary and setup of the app is done. So checking for meaning without opening the app atleast once might be causing this.

can we download it in the BG upon notification trigger? or can we just add a message to the user that the downloading is not yet completed?

tirkarthi commented 1 year ago

I would lean towards later as ProcessTextActivity is meant to be instant and I don't want to add more to it. Maybe a message or toast to open app or to show database missing is appropriate.