search

Serubin / pulse-sms-web

The official web app for Pulse SMS - built on Vue.js.
https://pulsesms.app
Other
111 stars 44 forks source link

Update Dependencies #153

Closed Serubin closed 3 years ago

Serubin commented 3 years ago

Description

Update dependencies to latest version - general clean up of deps.

Includes some duplicate work from #151 - may require a simple rebase once one or the other gets merged.

Serubin commented 3 years ago

@mohsintahir @MapleSimon What's the process for getting a review?

mohsintahir commented 3 years ago

@Serubin I think you can create a PR and assign it to me as reviewer if you want a review.

Serubin commented 3 years ago

@mohsintahir I can't do anything but submit PRs/Issues. Maple removed my access

mohsintahir commented 3 years ago

@Serubin Can you kindly mention me in comment to review something if you want to ?

Serubin commented 3 years ago

@mohsintahir Already done in all three PRs.

mohsintahir commented 3 years ago

@Serubin why have we removed core-js from dependencies?

Serubin commented 3 years ago

It's not used and we have no need for these types of polyfills