musora-content-services 2.116.0 → 2.117.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+## [2.117.0](https://github.com/railroadmedia/musora-content-services/compare/v2.116.0...v2.117.0) (2026-01-13)
+
+
+### Features
+
+* **BEH-1442:** old method migration + course collection updates ([#673](https://github.com/railroadmedia/musora-content-services/issues/673)) ([24dd6bf](https://github.com/railroadmedia/musora-content-services/commit/24dd6bf6a1604ef3ee639979cf2ffab5abc05a24))
+
 ## [2.116.0](https://github.com/railroadmedia/musora-content-services/compare/v2.115.3...v2.116.0) (2026-01-12)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "2.116.0",
+  "version": "2.117.0",
   "description": "A package for Musoras content services ",
   "main": "src/index.js",
   "type": "module",

package/src/constants/membership-permissions.ts

@@ -0,0 +1,7 @@
+
+export const MEMBERSHIP_PERMISSIONS = {
+  free: 134,
+  base: 91,
+  plus: 92,
+  lifetime: 108,
+}

package/src/contentTypeConfig.js

@@ -62,6 +62,7 @@ export const DEFAULT_FIELDS = [
   "'permission_id': permission_v2",
   'child_count',
   '"parent_id": parent_content_data[0].id',
+  '"grandparent_id": parent_content_data[1].id',
 ]
 
 // these are identical... why
@@ -157,13 +158,12 @@ export const showsTypes = {
 }
 
 export const coachLessonsTypes = [
+  'course-collection',
   'course',
   'course-lesson',
   'coach-stream',
   'student-focus',
   'quick-tips',
-  'pack',
-  'semester-pack',
   'question-and-answer',
   'song-tutorial',
   'song-tutorial-lesson',
@@ -201,7 +201,11 @@ export const individualLessonsTypes = [
   ...studentArchivesLessonTypes,
 ]
 
-export const coursesLessonTypes = ['course', 'course-collection', 'guided-course']
+export const coursesLessonTypes = [
+  'course',
+  'course-collection',
+  'guided-course',
+]
 
 export const skillLessonTypes = ['skill-pack']
 
@@ -250,8 +254,7 @@ export const lessonTypesMapping = {
 export const getNextLessonLessonParentTypes = [
   'course',
   'guided-course',
-  'pack',
-  'pack-bundle',
+  'course-collection',
   'song-tutorial',
   'learning-path-v2',
   'skill-pack',
@@ -266,7 +269,7 @@ export const progressTypesMapping = {
     ...studentArchivesLessonTypes,
     'documentary-lesson',
     'live',
-    'pack-bundle-lesson',
+    'course-lesson'
   ],
   course: ['course'],
   show: showsLessonTypes,
@@ -274,7 +277,7 @@ export const progressTypesMapping = {
   songs: transcriptionsLessonTypes,
   'play along': playAlongLessonTypes,
   'guided course': ['guided-course'],
-  pack: ['pack', 'semester-pack'],
+  'course collection': ['course-collection'],
   'learning path': ['learning-path-v2'],
   'skill pack': skillLessonTypes,
   'jam track': jamTrackLessonTypes,
@@ -311,12 +314,12 @@ export const recentTypes = {
   lessons: [
     ...individualLessonsTypes,
     'course-lesson',
-    'pack-bundle-lesson',
     'guided-course-lesson',
     'quick-tips',
   ],
   songs: [...SONG_TYPES],
   home: [
+    ...skillLessonTypes,
     ...individualLessonsTypes,
     ...tutorialsLessonTypes,
     ...skillLessonTypes,
@@ -326,7 +329,7 @@ export const recentTypes = {
     'learning-path-v2',
     'live',
     'course',
-    'pack',
+    'course-collection',
   ],
 }
 
@@ -339,7 +342,6 @@ export const ownedContentTypes = {
     ...coursesLessonTypes,
     ...skillLessonTypes,
     ...entertainmentLessonTypes,
-    'pack',
   ],
   songs: [
     ...tutorialsLessonTypes,
@@ -396,9 +398,6 @@ export let contentTypeConfig = {
   'guided-course': {
     includeChildFields: true,
   },
-  'course-collection': {
-    individualLessonsTypes: true,
-  },
   course: {
     fields: [
       '"lesson_count": child_count',
@@ -415,6 +414,9 @@ export let contentTypeConfig = {
     ],
     slug: 'courses',
   },
+  'course-lesson': {
+    fields: [`"resources": ${resourcesField}`],
+  },
   download: {
     fields: [
       `"resource": ${resourcesField}`,
@@ -490,7 +492,7 @@ export let contentTypeConfig = {
     ],
     slug: 'play-alongs',
   },
-  pack: {
+  'course-collection': {
     fields: [
       '"lesson_count": coalesce(count(child[]->.child[]->), 0)',
       `"description": ${descriptionField}`,
@@ -516,24 +518,9 @@ export let contentTypeConfig = {
     slug: 'rudiments',
   },
   routine: {
-    fields: [`"description": ${descriptionField}`, 'high_soundslice_slug', 'low_soundslice_slug'],
+    fields: [`"description": ${descriptionField}`, 'soundslice_slug'],
     slug: 'routines',
   },
-  'pack-children': {
-    fields: [
-      'child_count',
-      `"resources": ${resourcesField}`,
-      '"image": logo_image_url.asset->url',
-      '"thumbnail": thumbnail.asset->url',
-      '"light_mode_logo": light_mode_logo_url.asset->url',
-      '"dark_mode_logo": dark_mode_logo_url.asset->url',
-      `"description": ${descriptionField}`,
-    ],
-    childFields: [`"description": ${descriptionField}`],
-  },
-  'pack-bundle-lesson': {
-    fields: [`"resources": ${resourcesField}`],
-  },
   foundation: {
     fields: [
       `"description": ${descriptionField}`,

package/src/index.d.ts

@@ -257,7 +257,6 @@ import {
 	buildEntityAndTotalQuery,
 	fetchAll,
 	fetchAllFilterOptions,
-	fetchAllPacks,
 	fetchBrandsByContentIds,
 	fetchByRailContentId,
 	fetchByRailContentIds,
@@ -267,6 +266,7 @@ import {
 	fetchCommentModContentData,
 	fetchContentRows,
 	fetchContentTypeCounts,
+	fetchCourseCollectionData,
 	fetchHierarchy,
 	fetchLearningPathHierarchy,
 	fetchLeaving,
@@ -280,7 +280,6 @@ import {
 	fetchNewReleases,
 	fetchOtherSongVersions,
 	fetchOwnedContent,
-	fetchPackAll,
 	fetchPackData,
 	fetchPlayAlongsCount,
 	fetchRecent,
@@ -479,7 +478,6 @@ declare module 'musora-content-services' {
 		extractSanityUrl,
 		fetchAll,
 		fetchAllFilterOptions,
-		fetchAllPacks,
 		fetchArtistBySlug,
 		fetchArtistLessons,
 		fetchArtists,
@@ -499,6 +497,7 @@ declare module 'musora-content-services' {
 		fetchContentPageUserData,
 		fetchContentRows,
 		fetchContentTypeCounts,
+		fetchCourseCollectionData,
 		fetchCustomerPayments,
 		fetchEnrollmentPageMetadata,
 		fetchFollowedThreads,
@@ -531,7 +530,6 @@ declare module 'musora-content-services' {
 		fetchNotifications,
 		fetchOtherSongVersions,
 		fetchOwnedContent,
-		fetchPackAll,
 		fetchPackData,
 		fetchPlayAlongsCount,
 		fetchPlaylist,

package/src/index.js

@@ -261,7 +261,6 @@ import {
 	buildEntityAndTotalQuery,
 	fetchAll,
 	fetchAllFilterOptions,
-	fetchAllPacks,
 	fetchBrandsByContentIds,
 	fetchByRailContentId,
 	fetchByRailContentIds,
@@ -271,6 +270,7 @@ import {
 	fetchCommentModContentData,
 	fetchContentRows,
 	fetchContentTypeCounts,
+	fetchCourseCollectionData,
 	fetchHierarchy,
 	fetchLearningPathHierarchy,
 	fetchLeaving,
@@ -284,7 +284,6 @@ import {
 	fetchNewReleases,
 	fetchOtherSongVersions,
 	fetchOwnedContent,
-	fetchPackAll,
 	fetchPackData,
 	fetchPlayAlongsCount,
 	fetchRecent,
@@ -478,7 +477,6 @@ export {
 	extractSanityUrl,
 	fetchAll,
 	fetchAllFilterOptions,
-	fetchAllPacks,
 	fetchArtistBySlug,
 	fetchArtistLessons,
 	fetchArtists,
@@ -498,6 +496,7 @@ export {
 	fetchContentPageUserData,
 	fetchContentRows,
 	fetchContentTypeCounts,
+	fetchCourseCollectionData,
 	fetchCustomerPayments,
 	fetchEnrollmentPageMetadata,
 	fetchFollowedThreads,
@@ -530,7 +529,6 @@ export {
 	fetchNotifications,
 	fetchOtherSongVersions,
 	fetchOwnedContent,
-	fetchPackAll,
 	fetchPackData,
 	fetchPlayAlongsCount,
 	fetchPlaylist,

package/src/services/content.js

@@ -11,7 +11,7 @@ import {
   fetchUpcomingEvents,
   fetchScheduledReleases,
   fetchReturning,
-  fetchLeaving, fetchScheduledAndNewReleases, fetchContentRows, fetchOwnedContent
+  fetchLeaving, fetchScheduledAndNewReleases, fetchContentRows, fetchOwnedContent, fetchCourseCollectionData
 } from './sanity.js'
 import {TabResponseType, Tabs, capitalizeFirstLetter} from '../contentMetaData.js'
 import {recommendations, rankCategories, rankItems} from "./recommendations";
@@ -19,6 +19,8 @@ import {addContextToContent} from "./contentAggregator.js";
 import {globalConfig} from "./config";
 import {getUserData} from "./user/management";
 import {filterTypes, ownedContentTypes} from "../contentTypeConfig";
+import {getPermissionsAdapter} from "./permissions/index.ts";
+import {MEMBERSHIP_PERMISSIONS} from "../constants/membership-permissions.ts";
 
 
 export async function getLessonContentRows (brand='drumeo', pageName = 'lessons') {
@@ -458,24 +460,33 @@ export async function getRecommendedForYou(brand, rowId = null, {
  *   .then(content => console.log(content))
  *   .catch(error => console.error(error));
  */
-export async function getLegacyMethods(brand) {
-
-  // TODO: Replace with real data from Sanity when available with permissions
-
-  return [
-    {
-      id: 1,
-      title: '2020 Method',
-      type: 'pack',
-      child_count: 12,
-    },
-    {
-      id: 2,
-      title: '2016 Foundations',
-      type: 'pack',
-      child_count: 12,
-    },
-  ]
+export async function getLegacyMethods(brand)
+{
+  const brandMap = {
+    drumeo: [241247],
+    pianote: [
+      276693,
+      215952 //Foundations 2019
+    ],
+    singeo: [308514],
+    guitareo: [333652],
+  }
+  const ids = brandMap[brand] ?? null;
+  if (!ids) return [];
+  const adapter = getPermissionsAdapter()
+  const userPermissionsData = await adapter.fetchUserPermissions()
+  const userPermissions = userPermissionsData.permissions
+  // Users should only have access to this if they have an active membership AS WELL as the content access
+  // This is hardcoded behaviour and isn't found elsewhere
+  const hasMembership = userPermissionsData.isAdmin
+    || userPermissions.includes(MEMBERSHIP_PERMISSIONS.base)
+    || userPermissions.includes(MEMBERSHIP_PERMISSIONS.plus)
+  const hasContentPermission = userPermissions.includes(100000000 + ids[0])
+  if (hasMembership && hasContentPermission) {
+   return Promise.all(ids.map(id => fetchCourseCollectionData(id)))
+  } else {
+    return []
+  }
 }
 
 /**
@@ -509,7 +520,7 @@ export async function getLegacyMethods(brand) {
  * @example
  * // Fetch owned content filtered by types
  * getOwnedContent('drumeo', {
- *   type: ['course', 'pack'],
+ *   type: ['course', 'course-collection'],
  *   page: 1,
  *   limit: 10
  * })

package/src/services/contentAggregator.js

@@ -20,7 +20,7 @@ import {getContentAwardsByIds} from "./awards/award-query.js";
  *   {} <- fetchLessonContent || on playback page (main window)
  *   in the examples below, dataField would be set to `children`
  *   [{id, children}, {id, children,}] <- getTabData || catalog Page
- *   {childen, } <- getPackData || pack index page
+ *   {children, } <- getCourseCollectionData || Course Collection index page
  *
  *
  * @param dataPromise - promise or method that provides sanity data

package/src/services/contentProgress.js

@@ -94,7 +94,7 @@ export async function getNavigateTo(data, collection = null) {
   collection = normalizeCollection(collection)
   let navigateToData = {}
 
-  const twoDepthContentTypes = ['pack'] // not adding method because it has its own logic (with active path)
+  const twoDepthContentTypes = ['course-collection'] // not adding method because it has its own logic (with active path)
   //TODO add parent hierarchy upwards as well
   // data structure is the same but instead of child{} we use parent{}
   for (const content of data) {
@@ -134,7 +134,7 @@ export async function getNavigateTo(data, collection = null) {
         const lastInteracted = await getLastInteractedOf(childrenIds, collection)
         const lastInteractedStatus = childrenStates[lastInteracted]
 
-        if (['course', 'pack-bundle', 'skill-pack'].includes(content.type)) {
+        if (['course', 'skill-pack'].includes(content.type)) {
           if (lastInteractedStatus === STATE_STARTED) {
             // send to last interacted
             navigateToData[content.id] = buildNavigateTo(
@@ -169,7 +169,7 @@ export async function getNavigateTo(data, collection = null) {
             collection
           )
           if (childrenStates[lastInteractedChildId] === STATE_COMPLETED) {
-            // TODO: packs have an extra situation where we need to jump to the next course if all lessons in the last engaged course are completed
+            // TODO: course collections have an extra situation where we need to jump to the next course if all lessons in the last engaged course are completed
           }
           let lastInteractedChildNavToData = await getNavigateTo(firstChildren, collection)
           lastInteractedChildNavToData = lastInteractedChildNavToData[lastInteractedChildId]

package/src/services/progress-row/rows/content-card.js

@@ -169,7 +169,7 @@ function getDefaultCTATextForContent(content, contentType) {
     if (contentType === 'lesson') ctaText = 'Revisit Lesson'
     if (contentType === 'song tutorial' || collectionLessonTypes.includes(content.type))
       ctaText = 'Revisit Lessons'
-    if (contentType === 'pack') ctaText = 'View Lessons'
+    if (contentType === 'course-collection') ctaText = 'View Lessons'
   }
   return ctaText
 }

package/src/services/sanity.js

@@ -639,8 +639,6 @@ export async function fetchAll(
     bypassStatusAndPublishedValidation = true
   } else if (type === 'lessons' || type === 'songs') {
     typeFilter = ``
-  } else if (type === 'pack') {
-    typeFilter = `&& (_type == 'pack' || _type == 'semester-pack')`
   } else {
     typeFilter = type ? `&& _type == '${type}'` : ''
   }
@@ -1163,50 +1161,6 @@ export async function fetchRelatedLessons(railContentId) {
   return await fetchSanity(query, false, { processNeedAccess: true })
 }
 
-/**
- * Fetch all packs.
- * @param {string} brand - The brand for which to fetch packs.
- * @param {string} [searchTerm=""] - The search term to filter packs.
- * @param {string} [sort="-published_on"] - The field to sort the packs by.
- * @param {number} [params.page=1] - The page number for pagination.
- * @param {number} [params.limit=10] - The number of items per page.
- * @returns {Promise<Array<Object>|null>} - The fetched pack content data or null if not found.
- */
-export async function fetchAllPacks(
-  brand,
-  sort = '-published_on',
-  searchTerm = '',
-  page = 1,
-  limit = 10
-) {
-  const sortOrder = getSortOrder(sort, brand)
-  const filter = `(_type == 'pack' || _type == 'semester-pack') && brand == '${brand}' && title match "${searchTerm}*"`
-  const filterParams = {}
-  const start = (page - 1) * limit
-  const end = start + limit
-
-  const query = await buildQuery(
-    filter,
-    filterParams,
-    await getFieldsForContentTypeWithFilteredChildren('pack'),
-    {
-      sortOrder: sortOrder,
-      start,
-      end,
-    }
-  )
-  return fetchSanity(query, true)
-}
-
-/**
- * Fetch all content for a specific pack by Railcontent ID.
- * @param {string} railcontentId - The Railcontent ID of the pack.
- * @returns {Promise<Array<Object>|null>} - The fetched pack content data or null if not found.
- */
-export async function fetchPackAll(railcontentId, type = 'pack') {
-  return fetchByRailContentId(railcontentId, type)
-}
-
 export async function fetchLiveEvent(brand, forcedContentId = null) {
   const LIVE_EXTRA_MINUTES = 30
   //calendarIDs taken from addevent.php
@@ -1283,6 +1237,25 @@ export async function fetchLiveEvent(brand, forcedContentId = null) {
 }
 
 /**
+ * Fetch the data needed for the CourseCollection Overview screen.
+ * @param {number} id - The Railcontent ID of the CourseCollection
+ * @returns {Promise<Object|null>} - The CourseCollection information and lessons or null if not found.
+ *
+ * @example
+ * fetchCourseCollectionData(404048)
+ *   .then(CourseCollection => console.log(CourseCollection))
+ *   .catch(error => console.error(error));
+ */
+export async function fetchCourseCollectionData(id) {
+  const builder = await new FilterBuilder(`railcontent_id == ${id}`).buildFilter()
+  const query = `*[${builder}]{
+    ${await getFieldsForContentTypeWithFilteredChildren('course-collection')}
+  } [0...1]`
+  return fetchSanity(query, false)
+}
+
+/**
+ * DEPRECATED: Use fetchCourseCollectionData
  * Fetch the data needed for the Pack Overview screen.
  * @param {number} id - The Railcontent ID of the pack
  * @returns {Promise<Object|null>} - The pack information and lessons or null if not found.
@@ -1293,11 +1266,7 @@ export async function fetchLiveEvent(brand, forcedContentId = null) {
  *   .catch(error => console.error(error));
  */
 export async function fetchPackData(id) {
-  const builder = await new FilterBuilder(`railcontent_id == ${id}`).buildFilter()
-  const query = `*[${builder}]{
-    ${await getFieldsForContentTypeWithFilteredChildren('pack')}
-  } [0...1]`
-  return fetchSanity(query, false)
+  return fetchCourseCollectionData(id)
 }
 
 /**