This patch changes the text in the UI in the "Use SSL?" now "Protocol Security" section. I think this improves the easy of use and clarity of the configuration screen.

CommentFileSizeAuthor
#6 fillpdf_1403112_text_fixed.patch1011 byteswizonesolutions
#5 fillpdf_ui_protocol_security.patch1010 bytesliam morland
#3 fillpdf_ui_protocol_security.patch1 KBliam morland
fillpdf_ui_protocol_security.patch1.02 KBliam morland

Comments

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented

I think this wording makes things more confusing for the average user. It's too detailed. Perhaps "Use HTTPS?" instead of "Use SSL?" could work and the section about "between your server and the remote service" could also be added.

What do you think?

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Status: Needs review » Needs work
liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Needs work » Needs review
StatusFileSize
new fillpdf_ui_protocol_security.patch1 KB

Yes, just HTTPS instead of SSL would probably be more clear. In any case, it is often TLS now, not SSL. I think "Use/Do not use HTTPS" is easier than Yes/No. Patch attached.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

Fixed "remove service" to say "remote service" and committed this. Needs backport.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new fillpdf_ui_protocol_security.patch1010 bytes

Backport attached. Apply after #1430390: Coding standards: remove whitespace.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Status: Needs review » Fixed
StatusFileSize
new fillpdf_1403112_text_fixed.patch1011 bytes

I had to make the changes manually. They are in commit 192f63e. Attached my version of the patch for reference.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit fd9c0db on 7.x-1.x, 7.x-2.x, 7.x-2.x-tests1, 7.x-1.x-ubercartbackporttest authored by Liam Morland, committed by wizonesolutions: 
    Issue #1403112: Clearer HTTPS text.