Correct the composer require drupal/stripe_pay instructions

Created on 6 September 2023, over 1 year ago

Problem/Motivation

The documentation currently instructs the user to require the wrong project vendor package name.

Proposed resolution

Correct the documentation.

๐Ÿ› Bug report
Status

Active

Version

1.1

Component

Documentation

Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States jcandan

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @jcandan
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States jcandan
  • Status changed to Fixed 26 days ago
  • ๐Ÿ‡ณ๐Ÿ‡ต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 in composer.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 listed drupal/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.

Production build 0.71.5 2024