Merge pull request #311 from vidhiii1711/fix/roadmap-filter-limit-validation

fix: roadmap filters returning 400 due to limit validation mismatch

Author: Sachinchaurasiya360

server/src/module/roadmap/roadmap.service.ts

@@ -90,12 +90,19 @@ export async function listPublishedRoadmaps(opts: {
   if (opts.level && opts.level !== "ALL_LEVELS") {
     where.level = opts.level as "BEGINNER" | "INTERMEDIATE" | "ADVANCED" | "ALL_LEVELS";
   }
-  if (opts.tag) {
-    andConditions.push({ tags: { has: opts.tag } });
-  }
-  if (opts.category) {
-    andConditions.push({ tags: { has: opts.category } });
-  }
+  // if (opts.tag) {
+  //   andConditions.push({ tags: { has: opts.tag } });
+  // }
+  // if (opts.category) {
+  //   andConditions.push({ tags: { has: opts.category } });
+  // }
+  const tagFilters: string[] = [];
+if (opts.tag) tagFilters.push(opts.tag);
+if (opts.category) tagFilters.push(opts.category);
+
+if (tagFilters.length > 0) {
+  andConditions.push({ tags: { hasSome: tagFilters } });
+}
   if (opts.search) {
     const s = opts.search.trim();
     if (s) {

server/src/module/roadmap/roadmap.validation.ts

@@ -49,7 +49,8 @@ export const pdfThemeQuery = z.object({
 
 export const listQuerySchema = z.object({
   page: z.coerce.number().int().min(1).default(1),
-  limit: z.coerce.number().int().min(1).max(50).default(20),
+  // limit: z.coerce.number().int().min(1).max(50).default(20),
+  limit: z.coerce.number().int().min(1).max(100).default(20),
   level: z.enum(["BEGINNER", "INTERMEDIATE", "ADVANCED", "ALL_LEVELS"]).optional(),
   search: z.string().optional(),
   tag: z.string().optional(),