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

feat: Add command twill:refresh-crops #1289

Merged
merged 4 commits into from
Dec 6, 2021
Merged

feat: Add command twill:refresh-crops #1289

merged 4 commits into from
Dec 6, 2021

Conversation

pboivin
Copy link
Contributor

@pboivin pboivin commented Dec 2, 2021

Description

This complements #1258 and adds a new twill:refresh-crops Artisan command. The command takes a model name (FQCN or short name) and a role name. It then scans the existing mediables records to determine which crops are to be created or deleted. It supports a --dry flag to get an overview of the modifications to be made without actually touching the DB.

Related Issues

Related to #1258

Addresses #1193

Copy link
Member

@ifox ifox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is amazing @pboivin, thank you so much. I've added a comment for your review.

@pboivin pboivin merged commit 64303c1 into area17:2.x Dec 6, 2021
@pboivin pboivin deleted the feat/add-command-refresh-crops branch December 6, 2021 20:52
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.

2 participants