Skip to content

Reimplement trust banana display in profile to Vue

Chris Oelmueller requested to merge vue-banana into master

Release notes text

(added to !1795 (merged))

What does this MR do?

Reimplement trust banana display in profile to Vue

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

the move was rather straightforward. beta-testing should uncover the remaining problems, if some exist.

old data handling might be a problem if we have a prolonged Beta period. there's a possibility that injecting tags is possible on Beta which then get delivered and rendered to the old (PHP-view) Production code until next release.

Links to related issues

How to test

  • Check list of given bananas, layout has changed a bit there
  • Especially test giving new bananas (new frontend + changed backend)

Screenshots

image

image

Checklist

  • unmark as Draft (needs clarification for possible XSS impact, see above)
  • simple API test would be good to have later on
  • 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
  • added an entry to CHANGELOG.md
  • add a short text that can be used in the release notes
  • Once your MR has been merged, you are responsible to create a testing issue in Beta Testing Repo
Edited by Chris Oelmueller

Merge request reports