Skip to content
This repository was archived by the owner on May 7, 2024. It is now read-only.

Pull master into prod #58

Merged
merged 7 commits into from
Sep 13, 2023
Merged

Pull master into prod #58

merged 7 commits into from
Sep 13, 2023

Conversation

sopyb
Copy link
Owner

@sopyb sopyb commented Sep 13, 2023

No description provided.

sopyb and others added 7 commits September 13, 2023 19:59
The package-lock.json files for both the frontend and API were updated to increment the project version from 2.0.0 to 2.0.1.
…rieval

A new method, getRandomRoadmapId, was added to ExploreDB utility. This retrieves a random roadmap id from the database that is both public and non-draft. The change aids in efficient retrieval and sorting of the public, non-draft roadmaps, optimizing the performance with limited use of RAND() function.
The getRandomRoadmapId function in the ExploreDB class has been adjusted to better handle potential errors that could arise from the database query. Now, the function's return type is specific (Promise<bigint | null>), and explicit checks for additional conditions regarding the result of the query have been added. The aim is to reduce the chance of unhandled exceptions.
Implemented a new 'Feeling Lucky' search feature that returns a random roadmap. Additionally, response handling for the roadmap search was refactored to a separate helper file for cleaner code and better separation of concerns. This also reduces redundancy and makes the code more maintainable. The changes encompass updates in controllers, routes, and database utilities.
The versions of the "navigolearn-front" and "navigo-learn-api" are updated from "2.0.1" to "2.0.2" in package.json and package-lock.json files. This version bump reflects the recent additions to the application, particularly the addition of the 'feeling lucky' functionality to the Explore feature.
@sopyb sopyb merged commit 46d1cef into prod Sep 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant