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

ENT-7544: Added new app academy with related models #705

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

saleem-latif
Copy link
Contributor

@saleem-latif saleem-latif commented Oct 30, 2023

Description

An Academy backend is composed of multiple catalogs of learning content mapped as a single level tree of nodes. Each node stores an enterprise catalog as a definition of content included in the node.
An academy will be associated with content metadata similar to how an enterprise is associated with content metadata.

Ticket Link

Academies app in enterprise catalog

Post-review

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

@saleem-latif saleem-latif self-assigned this Oct 30, 2023
@saleem-latif saleem-latif force-pushed the saleem-latif/ENT-7544 branch 3 times, most recently from bb0d990 to 82e2abc Compare October 30, 2023 11:57
enterprise_catalog/apps/academy/models.py Outdated Show resolved Hide resolved
enterprise_catalog/apps/academy/models.py Outdated Show resolved Hide resolved
@saleem-latif saleem-latif force-pushed the saleem-latif/ENT-7544 branch from 82e2abc to 1175a58 Compare November 2, 2023 07:32
@saleem-latif saleem-latif force-pushed the saleem-latif/ENT-7544 branch from 1175a58 to 5d54573 Compare November 2, 2023 08:00
Copy link
Contributor

@irfanuddinahmad irfanuddinahmad left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@muhammad-ammar muhammad-ammar left a comment

Choose a reason for hiding this comment

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

LGTM

@saleem-latif saleem-latif merged commit 43a4d17 into master Nov 2, 2023
5 checks passed
@saleem-latif saleem-latif deleted the saleem-latif/ENT-7544 branch November 2, 2023 09:22
irfanuddinahmad pushed a commit that referenced this pull request Jul 24, 2024
ENT-7544: Added new app academy with related models
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.

3 participants