-
Notifications
You must be signed in to change notification settings - Fork 811
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
Notifications are disabled - Windows 10 #2844
Comments
I have a same issue, please help, how to enable this notification with CMD?. |
Same issue! Help |
Hi, I have the same Issue. First of all if you didn't want notifications and disabled them via Windows u can just add As I wanted to get the notifications back I tried to get two things.
This had no effect the notification were still hidden.
In the Registry Editor i went to Any Ideas on how to fix this would be really appreciated. Thanks in advance! |
+1 I encountered this issue today. I was tired of hearing the audio of the notification, so I just disabled the notification entirely via the notification's window cog icon, but after I did that, I encountered the same error you guys have. I've tried different things over the span of multiple hours, but not dice unfortunately... I tried deleting the "Enabled" property in the "Laravel Mix" notification key in the registry editor + reboot. I tried removing the key entirely + reboot. I also tried removing "Snore.DesktopToasts" + reboot. I also found references in a Backup folder. Deleted those also, but still not working. I think the issue might be due to the fact an AppID is specified (Laravel Mix), when Laravel Mix is not an actual app, but that's a huge guess. I tested a project that uses Laravel Mix 5 and it seems like back then it wasn't an issue because no AppID was specified (you can see "SnoreToast" at the bottom of the notification instead of "Laravel Mix"). If I run
However, if I use the same command with "Laravel" or "Mix" or "Laravel Mix", it finds no app matching that (which makes sense because Mix has no executable file). |
Some links that could be useful: (in the last link, scroll a bit and checkout out the "Windows 10 Fall Creators Update (Version 1709) Note" section) |
Update (still no fix found): I cloned SnoreToast's project and tried debugging what was happening. So I found that bit of code: std::wstring error;
NotificationSetting setting = NotificationSetting_Enabled;
if (!ST_CHECK_RESULT(d->m_notifier->get_Setting(&setting))) {
tLog << "Failed to retreive NotificationSettings ensure your appId is registered";
}
switch (setting) {
case NotificationSetting_Enabled:
ST_RETURN_ON_ERROR(setEventHandler(d->m_notification));
break;
case NotificationSetting_DisabledForApplication:
error = L"DisabledForApplication";
break;
case NotificationSetting_DisabledForUser:
error = L"DisabledForUser";
break;
case NotificationSetting_DisabledByGroupPolicy:
error = L"DisabledByGroupPolicy";
break;
case NotificationSetting_DisabledByManifest:
error = L"DisabledByManifest";
break;
}
if (!error.empty()) {
std::wstringstream err;
err << L"Notifications are disabled\n"
<< L"Reason: " << error << L" Please make sure that the app id is set correctly.\n"
<< L"Command Line: " << GetCommandLineW();
tLog << err.str();
std::wcerr << err.str() << std::endl;
} So what happens is the following:
|
Having the same issue! |
@piljac1 @bzioni
i know this is not a true way but it works :)) (i guess problem is deeply from SnoreToast). and i did it because of this first solution in Common Issues section in this page and btw if after this you still couldn't see mix notifications try rollback what you did in windows registery, by doing this:
|
@alikashfi It is something I tried as I stated in my latest reply:
In my case, I tried running the command manually with "Laravel Fake Mix" instead of "Laravel Mix", which worked as you also tried. However, to me it is not a viable fix. It might be for some that really want it to "work" for now though. |
hi, i just found my solution for this error. im using laravel btw
note : you have to do this for seperate projects you have, but well, it's better than nothing right. hope this solution works for you too guys |
Hmm none of this is ideal. I really need to look into this on my Windows machine. I'll do that this weekend and see if I can figure out a good solution. |
@thecrypticace That's great ! Let us know how it goes, because the suggested "modify code in your node_modules" solutions are not viable solutions in my book. Let me know if you need a hand. I spent hours on trying to find a fix and tried multiple things, but came up empty, but I'd be glad to take another look. |
I did a small amount of digging yesterday and it seems a lot of this was caused by the Fall Creators Update to Windows 10. It appears that we'll have to issue that installation command once whenever someone uses Mix. There's a few things I'm not yet sure of though:
Also, it appears that the app will be added to the user's start menu app list and I believe that is required. :( |
I have overcome this issue by SnoreToast notification On. |
@Rakibul8001 can you share more details or steps to solve it? |
@alikashfi Your solution worked for me thank you |
Finally, the permanent solution,
And it is..... :) I hope it helps everyone. |
@thecrypticace I think this issue is easy to solve, why not have an option to change the App ID that gets passed to the WebpackNotifierPlugin! |
If a collaborator can change the line below it should work on all projects: return new WebpackNotifierPlugin({
appID: 'Snore.DesktopToasts',
...
}); |
I'll give that a test today. |
I remembered why we "can't" do that — because notifications are only attributed to applications installed in the start menu on windows now and that's how the title is determined. Which attributes the title as "SnoreToast" instead of "Laravel Mix". So the only true solution is to run the install command if the app doesn't exist in the start menu. I would very much like some other option though because that's really inconvenient to have to deal with. |
Not sure what you mean, SnoreToast will have to be "installed" and in the start menu - but just once. The node-notifier package has a method |
It was specifically updated to handle this issue: #2671 But I think changing it back is probably best. sigh I'll do a few tests and make the update later this evening. Thanks. |
The only way I can get this error to happen is:
If the App ID is anything else it doesn't show up in notification settings, they cannot be disabled, and notifications work fine. I'd like to properly solve this but I need to be able to reliably reproduce the problem in the current configuration to be sure any changes we make actually fix them. Can some of you provide OS version details because as it stands this issue seems like it doesn't exist in Windows 11. So maybe it's a windows 10 only problem now. |
It works Perfectly.. |
Great!, this worked for me, I used DB Browser for SQLite to change wpndatabase.db. |
This also worked for me, Thank you Sensei, I used DB Browser to edit wpndatabase.db |
Hello, How to edit wpndatabase.db in HeidiSQL.? I've installed HeidiSQL too. but I don't see option to open file. How else we can delete row primaryId of Laravel Mix? |
Click the |
Thanks all, I am able to delete the row using DB Browser. And it it working now. |
Yes, this solution thanks bro |
Perfect! This worked for me after turning off the Laravel Mix notifications (as I thought I could turn it back on when needed, a big mistake). To get the Laravel Mix notifications back, I just deleted the whole Laravel Mix Windows register entry and the Laravel Mix row in the wpndatabase.db file using DB Browser (remember to save after deleting the db entry). I then ran npm run watch and saved a file, then a new windows reg and wpndatabase.db rows was created and my Laravel mix messages are back once again. Thanks Sensei |
thankyou so much alikashfi it work for me and safe my FYP |
This worked perfectly, thanks man you have saved me from serious headache |
Worked here. Thank you soo much. By default, Laravel Mix will display a system notification for each compilation. mix.disableNotifications(); Maybe you only want to be notified if there are any errors? That is possible too, by only disabling the success notifications. Please note that this is intended to be used via the watch command. The first compilation always displays a success notification. Only subsequent success notifications are disabled. mix.disableSuccessNotifications(); |
Funcionó para mí👌🏽 |
Tank you!, this this worked for me, |
Funciono |
npm list --depth=0
)node -v
): 15.8.0npm -v
): 7.5.1Description:
Laravel build notifications got disabled by itself - not sure what exactly happened and now when I run 'npm run watch' in my project folder I get the following message.
Notifications are disabled
Reason: DisabledForApplication Please make sure that the app id is set correctly.
Command Line: C:\wamp64\www\sterlingsilver\wp-content\themes\woocommerce-theme\node_modules\node-notifier\vendor\snoreToast\snoretoast-x64.exe -appID "Laravel Mix" -pipeName \.\pipe\notifierPipe-abb01a42-3adc-468c-a18a-8e2066933787 -p C:\wamp64\www\sterlingsilver\wp-content\themes\woocommerce-theme\node_modules\laravel-mix\icons\laravel.png -m "Build successful" -t "Laravel Mix"
I've tried uninstalling node.js, removing nodejs directories, removing nodejs from Path variable and reinstalling nodejs but to no avail.
npm -v = 7.5.1
node -v = 15.8.0
OS = Windows 10
Steps To Reproduce:
Following are my webpack.mix.js and package.json file content
webpack.mix.js --
const mix = require('laravel-mix');
mix.js('src/app.js', './')
mix.sass('src/app.scss', './',[])
mix.options({
processCssUrls: false
});
mix.sourceMaps()
package.json --
{
"private": true,
"scripts": {
"dev": "npm run development",
"development": "mix",
"watch": "mix watch",
"watch-poll": "mix watch -- --watch-options-poll=1000",
"hot": "mix watch --hot",
"prod": "npm run production",
"production": "mix --production"
},
"devDependencies": {
"axios": "^0.21",
"laravel-mix": "^6.0.11",
"lodash": "^4.17.19",
"mini-css-extract-plugin": "^1.3.6",
"postcss": "^8.1.14",
"resolve-url-loader": "^3.1.2",
"sass": "^1.32.7",
"sass-loader": "^8.0.2"
},
"dependencies": {
"bootstrap": "^4.6.0",
"cross-env": "^7.0.3",
"jquery": "^3.5.1",
"npm": "^7.5.4",
"popper.js": "^1.16.1",
"postcss-loader": "^5.0.0"
}
}
The text was updated successfully, but these errors were encountered: