Skip to content

Conversation

@uo294039
Copy link

@uo294039 uo294039 commented May 3, 2025

No description provided.

@uo294039 uo294039 added the documentation 📝 Improvements or additions to documentation label May 3, 2025
@uo294039 uo294039 self-assigned this May 3, 2025
Copy link

@Y4vra Y4vra left a comment

Choose a reason for hiding this comment

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

Several changes

@uo294039 uo294039 requested a review from Y4vra May 3, 2025 18:24
Copy link

@Y4vra Y4vra left a comment

Choose a reason for hiding this comment

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

A bunch of typos

- Strategy: This pattern will be used to define a family of algorithms, encapsulate each one, and make them interchangeable. This will help to promote modularity and maintainability. For example, it will be used to define different strategies for generating questions and answers, and for providing hints to the user.
- Template: This pattern may be used to define the skeleton of an algorithm in any superclass, letting the subclasses override specific steps of the algorithm without changing its structure. This will help to promote code reuse and extensibility, and can be used in different concerns of the application. For example, when defining the different types (categories) of questions or the game modes available in the application.
- Auth Service: It will manage the user login and the creation of the user token.
- User Service: It will add the user to the database.
Copy link

Choose a reason for hiding this comment

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

substitute for "It will manage the signup of new users into the system"

@uo294039 uo294039 requested a review from Y4vra May 3, 2025 21:51
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 3, 2025

Copy link

@Y4vra Y4vra left a comment

Choose a reason for hiding this comment

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

Looks great!

@Y4vra Y4vra merged commit 90682cb into master May 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📝 Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants