Skip to content

Convert management command to address licensing metadata issues#5760

Draft
bjester wants to merge 1 commit intolearningequality:hotfixesfrom
bjester:nsync-back-back-back
Draft

Convert management command to address licensing metadata issues#5760
bjester wants to merge 1 commit intolearningequality:hotfixesfrom
bjester:nsync-back-back-back

Conversation

@bjester
Copy link
Member

@bjester bjester commented Mar 16, 2026

Summary

Converts the management command that was previously used to export data for analyzing the issue of incomplete licensing metadata for nodes that have missing source nodes that were in public channels.

  • Uses a CSV for applying licensing metadata on a per-channel basis, all for public channels
  • Limits usage of .iterator() to when it handles a channel with imported nodes, to reduce the duration of how long a transaction is open

References

Reviewer guidance

AI usage

  • Changes to the management command were implemented, then AI was instructed to create tests for LicensingFixesLookup. Those tests were refactored to be more robust, then AI was told to update the management command tests for the same scenarios. Corrections were made to thoses tests until they passed.

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.

1 participant