wojciech.kapcia@tigase.net opened 3 years ago
|
|
I've added a check looking for a |
|
I checked it and following changes should also be made:
Optional: a) can we inline AGPL licence in the main installer page? |
|
@wojtek Input field is required as it's value is required on the server side - this way server decides if you can move on or cannot. I would prefer not to change that and "enforcing" acknowledgement is not a bad this. As for inlining - we can, but I thought keeping the installer simple would be better, and being able to print "license" is a nice thing (it is easy as it is a separate page). |
|
Well, then maybe at least changing the wording for FOSS version from "To confirm please enter your name or company name below" to something like "Type 'agree' to continue" or somesuch?
Pondering this use case - have you ever printed the licence text? Do you print a lot of things nowadays, considering that you can make virtually anything "digital"? The installer page can still be simple, with additional dive and correctly set overflow so the licence text would be scrollable without pushing down the form fields IMHO. |
|
Well, you are right. I do not print but I recall saving licenses in form of a PDF and sending them for review to someone making a decision if we can use the library or software under the following license.
Ok, I will use something like iframe. I still think that the whole license in the setup form is not a good idea, but that is just my opinion. |
|
I pondered it a little bit more and you might be right with presenting the licencing terms to "higher power" though in this case plain "AGPL" should be enough in that case. And if "printed" form is required then new page seems ok. Let's leave it as is and just adjust titles and wording in the form. |
|
As requested, I've removed the field to enter the company name from FOSS version (we are not used this value, so I've replaced it with |
|
Looks good now. |
Type |
Bug
|
Priority |
Normal
|
Assignee | |
Version |
Candidate for next minor release
|
As per parent task: