angular-localForage icon indicating copy to clipboard operation
angular-localForage copied to clipboard

feat: Bump localforage dependency to 1.5

Open NeverwinterMoon opened this issue 7 years ago • 5 comments

There are a few things that make 1.5 more desirable over 1.4 - one being this one https://github.com/localForage/localForage/pull/633. Then again, 1.5 has one breaking change: https://github.com/localForage/localForage/releases/tag/1.5.0

This affects only the projects that use npm packaging (browserify/webpack)

NeverwinterMoon avatar Nov 03 '17 09:11 NeverwinterMoon

@NeverwinterMoon

I've considered this same thing, actually. However, I feel like the Safari breaking change is enough to keep me from wanting to do this without bumping our major version number, which I would want to bundle up with some other breaking changes we've been considering.

I'd be open to accepting this PR if we also incorporated https://github.com/localForage/localForage-compatibility-1-4 so to avoid the breaking change for now. What do you think?

scotttrinh avatar Nov 03 '17 14:11 scotttrinh

@scotttrinh This does make sense to me, sure. I will try to incorporate the compatibility code and come back with the adjusted PR. Can't give any estimations yet as it's a bit hectic at work now.

NeverwinterMoon avatar Nov 06 '17 07:11 NeverwinterMoon

Thanks, @NeverwinterMoon! Let me know if there is anything I can do to support you.

scotttrinh avatar Nov 06 '17 12:11 scotttrinh

Any plans to make this happen?

mcarriere avatar Jan 15 '21 16:01 mcarriere

@mcarriere Nothing currently in the works, but I'd be happy to help with anyone who wants to take a stab at a PR. I no longer use it for my daily work, so I'd hope it would be driven mostly from people who do use it.

scotttrinh avatar Jan 15 '21 16:01 scotttrinh