-
-
Notifications
You must be signed in to change notification settings - Fork 437
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
Link to NotifyStock feed is shown only if enabled #3520
Conversation
thanks guys, I had to fix a phpcs error (which I wasn't expecting actually, and looks weird to me now, but anyway) |
I'll merge it just to get rid of it since it was actually approved before the phpcs whitespace fix (which I don't like anyway but... it's phpcs...) |
@fballiano The same problem occurs in the following files:
|
ehheheh sorry :-) if you can check that it works for you too and leave a review I can try to merge it quickly |
This link:
should appear only if the RSS is actually enabled.
This PR should fix this.