Fix: hotfixes #337
Merged
Fix: hotfixes #337
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shivraj-bhagat
added a commit
that referenced
this pull request
Feb 14, 2025
This reverts commit 43ec36a.
Merged
shivraj-bhagat
added a commit
that referenced
this pull request
Feb 14, 2025
shivraj-bhagat
added a commit
that referenced
this pull request
Feb 14, 2025
* Updated Header Profile Image locator and added test scripts for My Learning Paths and Approvals Page. (#294) - Test cases added for the profile image upload. * Enhance/newrelic (#317) * WIP: Integrate New Relic for application monitoring and logging * Enhance: Update New Relic configuration for improved monitoring and remove the deprecated file * Fix: LEARN-52 reduced Multiple calls and added a prefetch superLink (#318) * Fixed multiple calls on refresh * reset reactstrictMode * added prefetch in all Links * fixed system preferences error * Refactor: TeamMemberListing and TeamMemberDetails components for improved readability and performance; update Breadcrumb and Navbar for better key handling; modify RouteTab to use button element for accessibility. * Learn64: Add Flagged Feature (#312) * Feature done * refactor: add display_name to TUser type; enhance LessonProgressTypes with TDailyLessonResponse; update contentRepository with TFlaggedLesson; refactor toastUtils for error handling; adjust lesson service search logic; modify flagged lesson entity relationships * feat: add breadcrumb navigation to DailyLessonDetail; enhance Breadcrumb and ViewLesson components with link disabling functionality * refactor: add success message for lesson flagging; remove console log and simplify error handling in LessonController * fix: formatting --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * Chore: Update backend startup process with new script and permissions (#319) * Feat: Implement flagged lesson functionality with updates to DTOs, frontend components, and language strings (#320) * Fix: Update flagged lessons count to use totalLessons instead of flaggedLessons length (#321) * Fix: updated the unflagged count at first load for listing (#324) * Fix/multiple (#326) * Fix: update messages what is accepted in the field. * Fix: Enhance flagged lesson functionality with admin role checks and middleware updates * Fix: grammar issues * Fix: lint * Fix: streamline lesson update process by removing unnecessary deletion of flagged lessons * Fix: enhance selector usage and type safety across various slices and components * Fix: update Prettier configuration, improve type definitions, and refactor component function declarations * Fix: refactor lesson component to use selector for roadmap data and streamline state updates * Fix: lints * Refactor: Learn 56 Swagger documentation refactoring (#322) * fixes done * Refactor: Update constructor properties to be readonly and standardize API endpoint paths * Fix: update approval data fetch condition to include only MEMBER user type --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * Enhancement: LEARN-72 LEADERBOARD added with the cronjob and show weekly Ranking. (#325) * first commit * added backend logic for leaderboard * added Leaderboard services and frontend * added leaderboard ranking * Leaderboard cronjob added * linting fix * removed extra api * removed unused api * lint fix * Leaderboard is showing in every role now * changes as per feedback * changed weekly leaderboard email template as per feedback * cosmetic changes as per feedback * cosmetic changes as per feedback2 * rename leaderboard endpoint and remove unused pagination service --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * added quick fix (#327) * Fix: update lesson and course filtering to exclude archived entries (#328) * Enhancement: Learn76 Add pagination (#323) * fixes done * fixes * fixes * final fixes * Refactor lesson approval logic and improve skeleton loading components * Update approval list skeleton display logic for initial load * Fix query formatting in lesson search to improve readability --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * Fix: LEARN 88,89 Changes Capitalise to First letter capital (#329) * fixed capitalise to first letter capital * lint fix * changes as per feedback * Refactor: Update TeamMemberListing and ApprovalListSkeleton components for improved state management and rendering * Refactor: Improve approval list component by enhancing state management and updating system preferences handling * Enhancement: Add flagged lessons feature with pagination and state ma… (#332) * Enhancement: Add flagged lessons feature with pagination and state management * Refactor: Remove unused lesson approval and unflagging thunks from approval and flagged slices * Enhancement: Learn-70 Nest js11 version (#333) * fixes done * fixes * Refactor: Update API routes for lesson progress and user roadmaps for consistency * Refactor: Improve string interpolation for API endpoint construction in lessonsService --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * Enhancement: LEARN-91 monthly leaderboard added. (#330) * Monthly Leaderboard added * linting fix * Enhancement: Add LeaderboardTypeEnum and refactor leaderboard service to use it * changes as per feedback * changed button styling * changes as per feedback * fixed swagger problem * fix for cronjob dto * added linting fixes * quick fix * refactor: update leaderboard migration and improve leaderboard display logic * refactor: move date range functions to shared library and update imports --------- Co-authored-by: Shiv Raj Bhagat <shivraj.bhagat@crownstack.com> * changes as per feedbacks (#335) * Fix: hotfixes (#337) * changes as per feedbacks * Quick last changes in Ranking and navigating to the particular URL * cosmetic issue resolved * quick fix * quick changes * Revert "Fix: hotfixes (#337)" (#338) This reverts commit 43ec36a. --------- Co-authored-by: AnishkaR-byte <anishka.rai1@crownstack.com> Co-authored-by: Gagan Prakash <gagan@crownstack.com> Co-authored-by: Ujjwal Saini <74203184+Ujjwal164@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist