Module Summary

This is a simple module that allows site builders to easily embed the Chatlio live chat widget on their sites. Chatlio is a live chat service that allows stakeholders to easy engage with visitors via Slack, a popular collaborative chat client.

View the sandbox project page.

Clone the repo

git clone --branch 7.x-1.x-dev https://git.drupal.org/sandbox/richardrobinson/2678122.git chatlio

Notes

While I have written custom modules, this is my first time applying for a full project. I have done my best to meet the requirements in the Project Application Checklist. The only part I'm not sure on is 4.2 about 3rd party code. I included Chatlio's JavaScript for the widget, because Drupal retrieves settings from the database and injects them into the JavaScript.

Please let me know if you need anything else. I'm excited to get this up.

Comments

richardrobinson created an issue. See original summary.

richardrobinson’s picture

richardrobinson commented
Issue summary: View changes
richardrobinson’s picture

richardrobinson commented
Title: Chatlio » [D7] Chatlio
richardrobinson’s picture

richardrobinson commented
PA robot’s picture

PA robot commented

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

pankajsachdeva’s picture

pankajsachdeva commented

Automated Review

Found some issues on pareview.sh. Follow link:

Manual Review

Individual user account
[Yes: Follows] the guidelines for individual user accounts.
No duplication
[Yes: Does not cause] module duplication and/or fragmentation.
Master Branch
[No: Does not follow] the guidelines for master branch.
Licensing
[Yes: Follows] the licensing requirements.
3rd party assets/code
[Yes: Follows] the guidelines for 3rd party assets/code.
README.txt/README.md
[Yes: Follows] the guidelines for in-project documentation and/or the README Template.
Code long/complex enough for review
[No: Does not follow] the guidelines for project length and complexity.
Secure code
[Yes: Meets ]
Coding style & Drupal API usage
[List of identified issues:
  1. Provide proper config path 'admin/config/system/settings/chatlio' in .info file.
  2. Provide full detail in README.txt.
  3. Just a recommendation
  4. Configuration form should be in admin.inc file.
After reviewing code, I don't think that module will provide the desired functionality. It just contain a system setting form, hook_menu and a permission.

If added, please don't remove the security tag, we keep that for statistics and to show examples of security problems.

This review uses the Project Application Review Template.

pankajsachdeva’s picture

pankajsachdeva commented
Status: Needs review » Needs work
PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.