Skip to content

Remove obsolete warning code

Alex requested to merge 777-remove-obsolete-code into master

Closes #777 (closed)

What does this MR do?

Removes code for jsonp in xhrapp.php that returns a warning to users of the old app. Nick checked the logs (see https://yunity.slack.com/archives/C1T1ZM9A4/p1579047609055400) and there have been no jsonp requests in 2020 yet, so I think it can be safely removed.

How confident are you it won't break things if deployed?

Very sure. Doesn't affect the site or the newer apps.

How to test

I'm not sure if there's a reasonable way to test this. As long the site doesn't fail completely, everything should be fine.

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Johannes

Merge request reports