Hi guys,

Just a few more improvements that could probably be very easily added to the module:

By going through the project's page, and more particularly:

How to get a KEY and SECRET code?
For any authentication provider to be integrated, a KEY and SECRET code should be configured and these ones could be obtained on the following websites:

I was wondering if it would be possible to add direct links for each API provider on each CSNA provider's configuration form.

For example, currently, for the Kaixin CSNA settings form (csna_kaixin module), the description is:
Set your Kaixin authorized KEY and SECRET below. (See csna_kaixin.module, line 17).

Maybe we could improve the description a little bit by adding some more information about the corresponding websites where users could apply for the KEY and SECRET codes, something like:

Set your Kaixin authorized KEY and SECRET below.
If you want to use an API key, you can get one at the Kaixin API website.

Many different modules have similar help/description texts, such as GMap, WYSIWYG, or other modules.

I think these very small changes to sub-modules could slightly improve module's usability and get closer to Drupal's recommended best practices.

Please let me know if you would have any objections, questions, comments or issues on this request, I would surely be glad to explain in more details.

Thanks very much for your feedback in advance.
Cheers!

CommentFileSizeAuthor
#1 Patch for improving the description texts for all four sub-modules CSNA API providers settings form.3.46 KBdydave

Comments

dydave’s picture

dydave commented
Status: Active » Needs review
StatusFileSize
new Patch for improving the description texts for all four sub-modules CSNA API providers settings form.3.46 KB

Quick follow up on this ticket.

Please find attached to this comment a patch against the latest 7.x-1.x version 8866f85 which should fulfill the previous request in this ticket summary: csna-sub-modules-api-links-description-text-1908270-1.patch

Detailed log:
Modified all current four sub-modules (Kaixin, QQ, Renren, Weibo) providers form configuration description text to add all the links to third party API websites.

Patch has been tested and seems to work as expected.

Please let me know if you would have any questions, objections, comments, suggestions, recommendations or concerns on the patch or the request above, I would be glad to explain in more details or re-roll the patch if necessary.

I would greatly appreciate if you could take a look at the attached patch and potentially consider committing it to module's repository, it would certainly help improving overall module's usability and quality.

Thanks in advance for your feedback and testing.
Cheers!

alfababy’s picture

alfababy commented

Hi DYdave,

Thanks for your patch, this is good idea on settings form.

I patched your coding on 35b74fb.

alfababy’s picture

alfababy commented
Status: Needs review » Closed (fixed)