Skip to content
This repository has been archived by the owner on Mar 8, 2019. It is now read-only.

Major rewrite #47

Closed
wants to merge 18 commits into from
Closed

Major rewrite #47

wants to merge 18 commits into from

Conversation

Jibec
Copy link

@Jibec Jibec commented Aug 10, 2018

hi, here are more significant changes, to review after the other pull request.
work is still in progress and is an invitation to discuss together, not fully tested.

As of now, it doesn't work on my VM Debian 9 / YunoHost 3. But it has exactly the same behavior as your code: it does install but I can't connect.

Please do not merge before we both confirm it is working fine :)

@Jibec Jibec changed the title Majore rewrite Major rewrite Aug 11, 2018
@alexAubin
Copy link

(bump @src386 ? In case you don't want to maintain this app anymore, it can be maintained on behalf of YunoHost-Apps if there are people like @Jibec to volunteer)

@alexAubin
Copy link

Would be nice to know what to do with this, also considering that the app is now flagged as unmaintained ...

If this PR really does improve stuff it'd be nice to merge it :/

Could run this branch on the dev CI to see if there's no regression ...

@Jibec
Copy link
Author

Jibec commented Feb 17, 2019

indeed, it's been a long time now... can you please give set me as a maintainer @alexAubin ? It will take some days/weeks/months, but I'll finish this package and get people onboard.

@alexAubin
Copy link

I don't have write access to this repo :/ But maybe we could simply fork it to @YunoHost-Apps ?

@maniackcrudelis
Copy link

This app is no longer maintained, #51.
We do not have write permission on this repo.

Therefor, please redirect your PR to the fork in YunoHost-Apps, https://github.com/yunohost-apps/movim_ynh.

@Jibec
Copy link
Author

Jibec commented Feb 20, 2019

ij

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants