- Issue created by @jcandan
- Status changed to Fixed
26 days ago 2:04am 28 January 2025 - ๐ณ๐ตNepal sujan shrestha Kathmandu๐ณ๐ต, Nepal
Hi jcandan, Thank you for the feedback, and I apologize for the late response. Iโve updated the documentation as per your feedback regarding the composer require drupal/stripe_pay instructions. Thank you!
- ๐บ๐ธUnited States jcandan
My bad. I should have left a more clear instruction about what the proposed resolution should be.
The module correctly lists the
stripe/stripe-php
dependency incomposer.json
.Therefore, there is no need to have a developer additionally require that dependency.
In the original post, the thing I was pointing out was that to acquire this module, named
stripe_pay
, the instructions incorrectly listeddrupal/stripe
in 2 places.Most folks working with Drupal 8+ know they can acquire modules via Composer. However, if this instruction is to remain, the documentation should be updated to the the following:
Since the module requires an external library, Composer must be used.
composer require "drupal/stripe_pay"
...
Getting started
- Create a Stripe Account if you don't have one already.
- Download and install this module with composer require "drupal/stripe_pay"
- ...
...
Hope this clarification helps.