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

Coorrency provides your website the availability to convert currencies with super updated rates.

Based on Yahoo Finances responses, Coorrency brings (via ajax or not) the rate conversion ultra fast.

This module provides:

  • A page to be accessed directly from the browser, calculating the conversion. /coorrency/convert/from/to
  • A block that can be placed anywhere

The module provides a simple configuration, whether to use ajax or not. If yes, the results will be displayed right under the coorrency form,
otherwise it will redirect the user to /coorrency/convert/from/to.

Demo

Check out a demo at coorrency.com.

git clone --branch 8.x-1.x https://git.drupal.org/project/coorrency.git

CommentFileSizeAuthor
#3 2b79f5d69b.jpg23.36 KBsemjuel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rafaelferreir4 created an issue. See original summary.

PA robot’s picture

Issue summary: View changes
Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpsgitdrupalorgprojectcoorrencygit

Fixed the git clone URL in the issue summary for non-maintainer users.

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.

semjuel’s picture

FileSize
23.36 KB

Hi @rafaelferreir4

Automated Review

FAILED. Please check the result - https://pareview.sh/node/2109. You need to fix all issue from that report.

Manual Review, following template

Individual user account

Yes: Follows

No duplication

Yes: Follows, I haven't been able to find a similar project.

Master Branch

Yes: Follows the guidelines for master branch.

Licensing

Yes: Follows the licensing requirements.

3rd party assets/code

Yes: Follows.

README.txt/README.md

No: Does not follow. Please, see guidelines for in-project documentation

Secure code

Yes: Meets the security requirements.

Also, please, pay attention than class "SafeMarkup" is deprecated and will be removed .
And maybe it will be better if you add titles to fields (see attached).

PA robot’s picture

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.