-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
fix: copy lang management command - include PageUrl #7548
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
base: main
Are you sure you want to change the base?
Conversation
Include copy of related PageUrl models
@vasekch Thanks for this PR. Is there anything I can do to move it forward? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This will now be closed due to inactivity, but feel free to reopen it. |
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions. |
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7548 +/- ##
==========================================
+ Coverage 89.61% 89.63% +0.02%
==========================================
Files 129 129
Lines 12721 12738 +17
==========================================
+ Hits 11400 11418 +18
+ Misses 1321 1320 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@vasekch Do you think you can add a test to this PR? |
Description
Add PageUrl handling to CopyLangCommand. Fixes #8313
Currently using copy lang command breaks the site.
TODO add tests
Checklist
develop-4