This module places a Call Us button (click-to-call) to the different place in your website of the screen which is visible for your mobile visitors and website.
This module very useful for website face-time and mobile because your mobile visitors are already holding their phone it will allow them to call you with one simple touch of the button. No more complicated copy-and-pasting or memorizing the phone number!.
You can also add a button for the social link in website with the using of this module

Project link

https://www.drupal.org/project/callus

Git instructions

git clone --branch 8.x-1.x https://git.drupalcode.org/project/callus.git

Pareview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-callus.git

Comments

Pinesh Kumar created an issue. See original summary.

pinesh’s picture

Issue summary: View changes
vuil’s picture

Title: callus[D8] » [D8] Callus
Issue summary: View changes
Status: Needs review » Needs work

Thank you for the contribution!

Added a Pareview link into the issue summary.

(1.) Remove all .DS_Store files from your repository.

(2.) Update your README.txt file as the following requirements: https://www.drupal.org/node/2181737

DrupalPractice has found some issues with your code.
(3.)

FILE: ...1/web/vendor/drupal/pareviewsh/pareview_temp/src/Form/CallUsForm.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------
  69 | WARNING | t() calls should be avoided in classes, use dependency
     |         | injection and $this->t() instead
  71 | WARNING | #description values usually have to run through t() for
     |         | translation
 134 | WARNING | \Drupal calls should be avoided in classes, use
     |         | dependency injection instead
 189 | WARNING | Unused variable $config.
--------------------------------------------------------------------------

(4.)

FILE: .../site1101/web/vendor/drupal/pareviewsh/pareview_temp/callus.info.yml
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
--------------------------------------------------------------------------
 1 | WARNING | Remove "project" from the info file, it will be added by
   |         | drupal.org packaging automatically
 1 | WARNING | Remove "version" from the info file, it will be added by
   |         | drupal.org packaging automatically
--------------------------------------------------------------------------

(5.) There are also many issues into js/callup.js code.

avpaderno’s picture

Thank you for applying!
In addition to what reported in the previous comment, 8.x-1.x-dev is a wrong branch name; the correct branch name would be 8.x-1.x. The wrong branch needs to be removed.

pinesh’s picture

Issue summary: View changes
avpaderno’s picture

Issue summary: View changes

Also 8.x-1.0 is a wrong branch name, and it should be removed.

avpaderno’s picture

Issue tags: -Module review
pinesh’s picture

Thank you @ilchovuchkov and @kiamlaluno for the quick reply!
@kiamlaluno please check branch name.

pinesh’s picture

Status: Needs work » Needs review
vuil’s picture

Status: Needs review » Reviewed & tested by the community

Thank you for the contribution!

I have not found any security related issues into the code.
Please be patience through the whole security review process.
Thank you again.

avpaderno’s picture

Assigned: Unassigned » avpaderno
Status: Reviewed & tested by the community » Fixed

Thank you for your contribution! I am going to update your account.

These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

Status: Fixed » Closed (fixed)

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