Skip to content
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

OpenMage Feed Notifications in Backend #1507

Closed
ADDISON74 opened this issue Mar 24, 2021 · 6 comments
Closed

OpenMage Feed Notifications in Backend #1507

ADDISON74 opened this issue Mar 24, 2021 · 6 comments

Comments

@ADDISON74
Copy link
Contributor

OpenMage is using this URL for feeding notifications in Backend:

<feed_url>www.openmage.org/Web_Notifications/notifications_v1.rss</feed_url>

If you check this page content you will see an item from 2019, "Thank you for installing OpenMage M1 LTS!".

I suggest to use this feature and send periodically notifications related to OpenMage, at least for new releases and features. For example, there is a new feature related to ordering Terms & Conditions in checkout, we improved the speed for X, we fixed the Y.

@fballiano
Copy link
Contributor

I'd like to see the adminnotifications dead and buried 😅😅
But if it has to stay it should be used as you propose :-)

@fballiano
Copy link
Contributor

But I think this issue is a duplicate of #683

@alexh-swdev
Copy link
Contributor

alexh-swdev commented Apr 28, 2021

This seems to affect Amasty plugins. Due to parsing errors in the logs, I tried to deactivate the whole notifications plugin in Mage_All.xml yesterday and the backend of the store using Amasty Plugins was dead. Even if I deactivated the Amasty notifications.
The other one without those did still work.

I don't know if there are more similar coded plugins out there.

The affected store is still on OM 20.0.7

@fballiano
Copy link
Contributor

I usually just disable admin_notification output, instead of deactivating it, but we could simply create a module that fixes the amasty notification system...

@alexh-swdev
Copy link
Contributor

alexh-swdev commented Apr 29, 2021

I have the output disabled, but still had the parsing errors in the logfile.
By the way, Wyomind Watchlog also depends on admin notifications:
Wyomind_Watchlog_Helper_Data::checkNotification(){
// ...
$notify = Mage::getModel('adminnotification/inbox');
$item = $notify->getCollection()->addFieldToFilter('title', array("eq" => "Watchlog security
warning"))->addFieldToFilter('is_remove', array("eq" => 0));
//...
}

So I hope OM does not just remove it ;)

@ADDISON74
Copy link
Contributor Author

I close this issue. Please follow the discussions here #1655.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants