YunoHost-Apps / shaarli_ynh

Shaarli package for YunoHost
GNU General Public License v3.0
20 stars 13 forks source link

Update to 0.9.6 #30

Closed mkody closed 6 years ago

mkody commented 6 years ago

Tested with a fresh install (and with manual backup of data -> uninstall -> install -> restore data).

anmol26s commented 6 years ago

Not working on fresh install for me.

alexAubin commented 6 years ago

(Bump ? Is this a duplicate work of #29 ?)

mkody commented 6 years ago

29 was set as "long term" with a lot of rewrite, this PR is just about bumping to the upstream version.

I'll update to 0.9.3 either today or tomorrow soon (sorry, work took over) and re-check if the fresh install really isn't working.


Please wait for me to test it.

mkody commented 6 years ago

Tested fresh install with 0.9.3, it works fine. Backing old data, reinstalling it to 0.9.3 and restoring data also works.

alexAubin commented 6 years ago

Bumpity bump ? I'm not in the app team, but just randomly checking repositories and this looks like a cool app to keep up to date 👍

Rafi594 commented 6 years ago

Hi

I'll continue the refactoring of this app and upgrade to the latest version

Rafi594 commented 6 years ago

https://github.com/YunoHost-Apps/shaarli_ynh/pull/29

Rafi594 commented 6 years ago

This PR will be merged in 3 days if @lapineige don't review.

mkody commented 6 years ago

I might still update it to 0.9.6 either later today or tomorrow. 0.9.3 is a good spot since it has the latest security fix, but 0.9.4 brought interesting features and fixes.

lapineige commented 6 years ago

Let's update directly to 0.9.6, as it doesn't introduce significant changes to the upgrade process. (and has some cool additions indeed :)

mkody commented 6 years ago

Tested, it works great.
Since it doesn't have an upgrade script yet (https://github.com/YunoHost-Apps/shaarli_ynh/pull/29#issuecomment-386269357), we still have to do it that way:

lapineige commented 6 years ago

Yeah, we really need that backup script...

It works well. Let's merge this. It's ok for you @Rafi594 ?