Page MenuHomePhabricator

Spike: should toasts be configurable?
Open, Needs TriagePublic

Description

Right now we're doing toasts a little ad-hoc. We have precedents in paste check, add reference, etc.

Questions

  • Can we bring the existing toasts in various checks into a helper function in BaseEditCheck and define them by configuration?
  • Can we move success/failure toast calls out of act() entirely, perhaps providing an override interface?

Checklist