#55 Allow optional extra installed apps

Closed
jocelyn wants to merge 3 commits from jocelyn/jd-editable-installed-apps into FFDN/master
jocelyn commented 9 years ago

fix #52 relates #53

fix #52 relates #53
jocelyn commented 8 years ago
Owner

@zorun bump ?

@zorun bump ?
zorun commented 8 years ago
Owner

Yes sorry, I plan to test that after #57 (and merge both together).

Yes sorry, I plan to test that after #57 (and merge both together).
zorun commented 8 years ago
Owner

Well, in fact, this is independent from #57.

I have adapted your branch to the VPN app change, and added a few fixes. I have also disabled the DSL app by default, just like the VPN (I don't know if you use it for faimaison).

It's in the editable-installed-apps branch, can you have a quick look before I merge it?

Well, in fact, this is independent from #57. I have adapted your branch to the VPN app change, and added a few fixes. I have also disabled the DSL app by default, just like the VPN (I don't know if you use it for faimaison). It's in the `editable-installed-apps` branch, can you have a quick look before I merge it?
jocelyn commented 8 years ago
Owner

@zorun Reviewed and tested, seems fine, good job :-)

(btw, that implicit organization with optional apps in the root dir seems a bit messy to me, but that's a long-term question, clearly outside the scope of this PR).

@zorun Reviewed and tested, seems fine, good job :-) (btw, that implicit organization with optional apps in the root dir seems a bit messy to me, but that's a long-term question, clearly outside the scope of this PR).
zorun commented 8 years ago
Owner

Great, merged in 24cffdbae6. Thanks for the work!

Yes, optional apps should probably have their own git repositories (and could be available through pip).

Great, merged in 24cffdbae6a1cb7b100d3d8cf05bbbf938081441. Thanks for the work! Yes, optional apps should probably have their own git repositories (and could be available through pip).
Please reopen this pull request to perform merge operation.
Sign in to join this conversation.
Loading...
Cancel
Save
There is no content yet.