Skip to content

Temporary fix for temporary wallpost problem

Chris Oelmueller requested to merge fix-wallpost-beta into master

because of the release coming up, here's a hotfix / patch for removing wallpost entries from some legacy views.

What does this MR do?

Allow removing wallpost entries on some legacy screens again.
This was made impossible by changes related to the store-wall refactoring in !1690 (merged) and will be fixed "the right way" in !1772 (merged) along with other issues!

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

I'm very confident. Basically it only injects a JS function that the view code expects to be in global scope.

Links to related issues

How to test

  1. Log in as any user
  2. Go to own profile
  3. Post a wallpost
  4. Remove the wallpost

Checklist

  • better testing of this part of our code by UI tests would be really great, but this is just a quick patch restoring functionality so we can finish up the release :)
  • no unrelated changes
  • asked someone for a code review
  • set a "for:" label to indicate who will be affected by this change
  • use "state:" labels to track this MR's state until it was beta tested
  • no CHANGELOG.md entry needed, !1772 (merged) will cover this and more

Merge request reports