Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix digital post commands #166

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Fix digital post commands #166

wants to merge 2 commits into from

Conversation

rimi-itk
Copy link
Collaborator

@rimi-itk rimi-itk commented Apr 1, 2025

Fixes issue with os2forms-digital-post:test:send not being found due to constructor issues:

 [debug] Could not instantiate Drupal\os2forms_digital_post\Drush\Commands\DigitalPostTestCommands: Too few arguments to function Drupal\os2forms_digital_post\Drush\Commands\DigitalPostTestCommands::__construct(), 0 passed in /app/vendor/drush/drush/src/Runtime/ServiceManager.php on line 333 and exactly 4 expected [2.55 sec, 40.77 MB]

@rimi-itk rimi-itk marked this pull request as draft April 1, 2025 07:52
@rimi-itk rimi-itk force-pushed the hotfix/digital-post-commands branch from b83eb88 to 3a63ce2 Compare April 1, 2025 08:47
@rimi-itk rimi-itk force-pushed the hotfix/digital-post-commands branch from 3a63ce2 to 9ad3ea8 Compare April 1, 2025 09:16
@rimi-itk rimi-itk marked this pull request as ready for review April 1, 2025 09:35
@rimi-itk rimi-itk requested review from stankut, skifter and jekuaitk April 1, 2025 09:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@stankut stankut Awaiting requested review from stankut

@skifter skifter Awaiting requested review from skifter

@jekuaitk jekuaitk Awaiting requested review from jekuaitk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rimi-itk