Convert BaseCollection.fetch to return a promise.
Review Request #11629 — Created May 27, 2021 and submitted — Latest diff uploaded
This is the first of what is likely to be many steps to move our
codebase towards a more modern feel of using promises and
awaitfor asynchronous operations. In this change,
BaseCollection.fetchhas been changed, along with everything that
actually uses that including similar methods inside
ResourceCollection. The old callback usage is still available through
a helper which will wrap the returned promise, but causes a warning to
be logged to the console.
This also tweaks to our browserslist config in order to ensure that
awaitwon't get polyfilled out due to niche browsers
(primarily on mobile) that we really don't care about supporting.
- Ran js-tests.
- Manually tested data population on the review dialog and post commit