Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Ability to provide path to get the ops files from #29

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

mathewparet
Copy link

This enhancement provides an extra parameter to the operations:show and operations:process commands to allow user to specify the path the operations files can be loaded from.

@mathewparet
Copy link
Author

That's all the commits. I'm sorry for the mess of submitting multiple commits to this pull request after it was originally raised. This is the first time I'm working on a pull request. I've tested all possible scenarios I could think off and it works fine now.

Enhancements so far:

  1. Added --path option to operations:show and operations:process so that you can load operations from a custom path.
  2. Added --database option to operations:process so that you can specify the database connection against which the operations should be run.

…den while running `operations:process` command. This is useful, say in case you wish to implement operations for a multi-tenancy setup using `spatie/laravel-multitenancy`. Most multi-tenancy packages require you to implement a special interface for jobs so that they can be tagged as a tenant job. This change provides support for that so that the developer can extend the `OneTimeOperationProcessJob` job and add a the required implementation of the TenantAware (or similar) interface so that the jobs run in tenancy.
@mathewparet
Copy link
Author

Support for --job flag so that the process job class can be overridden while running operations:process command. This is useful, say in case you wish to implement operations for a multi-tenancy setup using spatie/laravel-multitenancy. Most multi-tenancy packages require you to implement a special interface for jobs so that they can be tagged as a tenant job. This change provides support for that so that the developer can extend the OneTimeOperationProcessJob job and add a the required implementation of the TenantAware (or similar) interface so that the jobs run in tenancy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant