GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 (edited) 9 minutes ago, BillOatman said: Same test please with debug on. Emby.Notifications.Discord102.dll.zip 55.37 kB · 1 download Attached. All same, this time added an episode instead of a movie. And pls do note that Test message IS going through, as it did from the start. embyserver.txt Edited May 17, 2021 by GrimReaper76 Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 9 minutes ago, GrimReaper76 said: Attached. All same, this time added an episode instead of a movie. And pls do note that Test message IS going through, as it did from the start. Yeah thats because its barfing trying to process whats in the queue from Emby, not talking to discord. And again please ... tracking down the exact issue. As always, debug on. Emby.Notifications.Discord103.dll.zip 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 (edited) New Media notification just came through now, for that last episode added. Although it ain't enriched as @ryzilla's screenshot: even though is marked in Plugin settings: Edit: I think that's my hourly Playback Reporting plugin New Media notification. Edited May 17, 2021 by GrimReaper76 Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 1 minute ago, GrimReaper76 said: New Media notification just came through now, for that last episode added. Although it ain't enriched as @ryzilla's screenshot: even though is marked in Plugin settings: Probably because of the issue its having processing the queue from Emby. We're getting there 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 7 minutes ago, BillOatman said: Yeah thats because its barfing trying to process whats in the queue from Emby, not talking to discord. And again please ... tracking down the exact issue. As always, debug on. Emby.Notifications.Discord103.dll.zip 55.54 kB · 0 downloads Here, same. Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 Just now, GrimReaper76 said: Here, same. No server log 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 Just now, BillOatman said: No server log Yeah, sorry. embyserver-63756871304.txt Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 11 minutes ago, GrimReaper76 said: Yeah, sorry. It seems to be making some assumptions on what "richer" data exists that may not always be true. Dialing in now what exactly the field is. Same, debug on please. Emby.Notifications.Discord104.dll.zip 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 4 minutes ago, BillOatman said: It seems to be making some assumptions on what "richer" data exists that may not always be true. Dialing in now what exactly the field is. Same, debug on please. Emby.Notifications.Discord104.dll.zip 55.72 kB · 0 downloads That's some great progress. As ALL my metadata and artwork is handled by 3rd party app and only parsed by Emby, maybe it ain't writing something plugin is looking for. embyserver.txt Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 13 minutes ago, GrimReaper76 said: That's some great progress. As ALL my metadata and artwork is handled by 3rd party app and only parsed by Emby, maybe it ain't writing something plugin is looking for. And same again please. Emby.Notifications.Discord105.dll.zip 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 3 minutes ago, BillOatman said: And same again please. Emby.Notifications.Discord105.dll.zip 55.85 kB · 0 downloads Running out of things to add. embyserver.txt Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 (edited) @GrimReaper76 What is your server name configured as for the plugin? And is ServerNameOverride true or false? Edited May 17, 2021 by BillOatman 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 Just now, BillOatman said: @GrimReaper76 What is your server name configured as for the plugin? Maybe no space? Lemme test. Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 (edited) 3 minutes ago, GrimReaper76 said: Maybe no space? Lemme test. Whats your emby server name itself? And the thing to try would be unchecking "Replace "Emby Server with your server name" and trying webhook name shouldnt matter as that seems to be working Edited May 17, 2021 by BillOatman 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 Just now, BillOatman said: Whats your server name itself? And the thing to try would be unchecking "Replace "Emby Server with your server name" and trying webhook name shouldnt matter as that seems to be working Server name itself is BLADE. Same as I changed Webhook name to, now. Still no joy, test message goes through, new media crashes server. embyserver-63756874552.txt Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 @GrimReaper76 uncheck "Replace Emby Server with your server name" and try, no need for server log on this one 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 (edited) 6 minutes ago, BillOatman said: @GrimReaper76 uncheck "Replace Emby Server with your server name" and try, no need for server log on this one In process. Edit: @BillOatman Man, you're awesome. HUGE thanks for going through all this. I'd like to drop you a beer, is there a place I could do that? Thanks again. Edited May 17, 2021 by GrimReaper76 Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 (edited) @GrimReaper76 Sweet Thanks for the offer, but maybe just help @ryzilla if he needs it. He seems to be missing some GUID in his configuration. I'll ask for some more help in a bit, I'll try and put some kind of fix in for the server name. @Luke It's doing this to get the server name, which he has set but apparently is coming through as null to the plugin. private readonly IServerConfigurationManager _serverConfiguration; public Notifier(ILogManager logManager, IJsonSerializer jsonSerializer, IServerConfigurationManager serverConfiguration, ... _serverConfiguration = serverConfiguration; ServerConfiguration serverConfig = _serverConfiguration.Configuration; string serverName = options.ServerNameOverride ? serverConfig.ServerName : "Emby Server"; // Note: options.ServerNameOverride is true mediaAddedEmbed.embeds.First().title = $"{titleText} has been added to {serverName.Trim()}"; // serverName.Trim() throws null reference exception Is that not correct any more? Edited May 17, 2021 by BillOatman 1 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 5 minutes ago, BillOatman said: @GrimReaper76 Sweet Thanks for the offer, but maybe just help @ryzilla if he needs it. He seems to be missing some GUID in his configuration. I'll ask for some more help in a bit, I'll try and put some kind of fix in for the server name. @Luke It's doing this to get the server name, which he has set but apparently is coming through as null to the plugin. private readonly IServerConfigurationManager _serverConfiguration; public Notifier(ILogManager logManager, IJsonSerializer jsonSerializer, IServerConfigurationManager serverConfiguration, ... _serverConfiguration = serverConfiguration; ServerConfiguration serverConfig = _serverConfiguration.Configuration; string serverName = options.ServerNameOverride ? serverConfig.ServerName : "Emby Server"; // Note: options.ServerNameOverride is true Is that not correct any more? Sure, if he chimes in. I'm getting no more errors in log, so that's that. Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 @GrimReaper76 Try this with "Replace Emby Server with your server name" checked again. Debug log only if it fails please. I suspect you won't get your server name but it should at least not fail with it checked. After Luke tells me whats wrong I'll try and make the server name work and see about getting this change into github. Emby.Notifications.Discord107.dll.zip 2 Link to comment Share on other sites More sharing options...
GrimReaper 3293 Posted May 17, 2021 Share Posted May 17, 2021 28 minutes ago, BillOatman said: @GrimReaper76 Try this with "Replace Emby Server with your server name" checked again. Debug log only if it fails please. I suspect you won't get your server name but it should at least not fail with it checked. After Luke tells me whats wrong I'll try and make the server name work and see about getting this change into github. Emby.Notifications.Discord107.dll.zip 55.86 kB · 1 download "Replace Emby Server with your server name" checked, content added, notification WITH Webhook name received, no errors thrown in log, all peachy. Great work, mate. Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 Just now, GrimReaper76 said: "Replace Emby Server with your server name" checked, content added, notification WITH Webhook name received, no errors thrown in log, all peachy. Great work, mate. Sweet thanks. Link to comment Share on other sites More sharing options...
BillOatman 500 Posted May 17, 2021 Share Posted May 17, 2021 On 5/13/2021 at 11:17 PM, rechigo said: I am sorry for been updating this plugin, I just have not had the time nor motivation to work on it anymore, and I kinda forgot it even existed to be totally honest.. If anybody would like to fork the Github repo to try and fix it up & submit some pull requests, I (and it would appear the community members as well) would very much appreciate it. I still have the Github repo linked to a Discord webhook so I'll see whenever a pull request has been started and I'll be able to review them & publish releases to this thread & Github releases. This plugin desperately needs to be rewritten, I have very little knowledge in C# so the code isn't very readable to say the least, I believe this was my first project that I ever worked on with C#. tldr; I have been too busy & lacking in motivation to work on this plugin recently. if anyone is interested in maintaining this plugin, please let me know via PMs. Hi: I think I just put in a pull request to get it back functioning again. 1 Link to comment Share on other sites More sharing options...
Legend1981 0 Posted May 17, 2021 Share Posted May 17, 2021 last version emby beta 4.6.0.47-beta or all beta 4.6 the plugin Emby Discord Notifications 1.0.7.9 dont work more people have this problem?????? Link to comment Share on other sites More sharing options...
Luke 37065 Posted May 17, 2021 Share Posted May 17, 2021 17 minutes ago, Legend1981 said: last version emby beta 4.6.0.47-beta or all beta 4.6 the plugin Emby Discord Notifications 1.0.7.9 dont work more people have this problem?????? Hi, the developer is investing his own time and energy into the plugin. Please help make it easier for him to help you by seeing how to report a problem. Thanks. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now