BUUPE / inquisitor

An end-to-end system for BU UPE recruitment season
https://upe.bu.edu/inquisitor/
MIT License
1 stars 1 forks source link

Apply destructuring to the apply.js class to make it more readable #132

Closed vitorvicente closed 2 years ago

vitorvicente commented 3 years ago

Might be worth using destructuring here to make this more readable in the future, i.e.

Promise.all([
      loadApplicationFormConfig,
      loadUserData,
      loadGeneralSettings,
      loadUserApplication,
    ]).then(([applicationFormConfig, userData, generalSettings, userApplication]) =>
        ....
    );
vitorvicente commented 2 years ago

Fixed in https://github.com/BUUPE/Inquisitor/tree/improvements/3.0