Skip to content

Add license#11

Merged
meppsilon merged 3 commits intomainfrom
license
Oct 20, 2021
Merged

Add license#11
meppsilon merged 3 commits intomainfrom
license

Conversation

@meppsilon
Copy link
Contributor

No description provided.

@meppsilon meppsilon requested a review from a team as a code owner October 20, 2021 18:47
@Gudahtt
Copy link
Member

Gudahtt commented Oct 20, 2021

This license is incompatible with the extension. Is this controller ever intended to be used on the extension?

@meppsilon
Copy link
Contributor Author

This license is incompatible with the extension. Is this controller ever intended to be used on the extension?

@Gudahtt I copied over the license from the swaps-controller. How is it incompatible with the extension?

@Gudahtt
Copy link
Member

Gudahtt commented Oct 20, 2021

It's incompatible in that the terms of the extension license conflict with it. The extension license allows usage under certain terms, but this license does not. Therefore to include this as part of the extension would make it impossible to use it under those terms.

In order for it to be compatible, we will want to use the same proprietary license of all of our client-side proprietary code. Or we can make the code open-source. That is the plan for the swaps controller - we're moving it to an open source license.

@meppsilon meppsilon merged commit 2e057b4 into main Oct 20, 2021
@Gudahtt
Copy link
Member

Gudahtt commented Oct 20, 2021

To follow up on the previous comments, we'll be updating the license later on to something compatible. We have merged this as-is temporarily.

@rekmarks rekmarks deleted the license branch October 20, 2021 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants