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: programs ONLY inherit UUIDs for catalogs common to ALL content within. #688

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

pwnage101
Copy link
Contributor

@pwnage101 pwnage101 commented Oct 2, 2023

ENT-7729

Companion PR to enable this feature in stage: https://github.com/edx/edx-internal/pull/9323

Description

Description of changes made

Ticket Link

Link to the associated ticket
Link title

Post-review

  • Squash commits into discrete sets of changes
  • Ensure that once the changes have been deployed to stage, prod is manually deployed

@pwnage101 pwnage101 marked this pull request as ready for review October 3, 2023 04:59
Comment on lines +729 to +744
This DAG represents the complete test environment:
┌────────────┐┌────────────┐┌────────────┐
│*course-1 ││*course-2 ││*course-3 │
│------------││------------││------------│
│in catalog-1││ ││ │
│in catalog-2││in catalog-2││ │
│in catalog-3││in catalog-3││in catalog-3│
│ ││in catalog-4││in catalog-4│
│ ││ ││in catalog-5│
└┬───────────┘└┬───────────┘└┬───────────┘
┌▽─────────────▽─────────────▽───────────┐
│*program-1 │
│----------------------------------------│
│(should inherit catalog-3 only) │
└────────────────────────────────────────┘
* = indexable
Copy link
Contributor

Choose a reason for hiding this comment

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

amazing

@pwnage101 pwnage101 merged commit 8adc14d into master Oct 3, 2023
6 checks passed
@pwnage101 pwnage101 deleted the pwnage101/ENT-7729 branch October 3, 2023 17:40
irfanuddinahmad pushed a commit that referenced this pull request Jul 24, 2024
feat: programs ONLY inherit UUIDs for catalogs common to ALL content within.
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