Skip to content

Add class description to controllers #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 26, 2024

Conversation

MysticFragilist
Copy link
Contributor

@MysticFragilist MysticFragilist commented May 27, 2024

Add class description on controllers, wiki still awaiting some more info coming soon!

Copy link

@MouradRUQAM MouradRUQAM left a comment

Choose a reason for hiding this comment

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

La documentation est globalement claire et permet de mieux comprendre le projet. Mis à part peut être l'enum State à refactoriser et quelques coquilles dans la documentation, c'est du très beau travail.

/// Example of use:
/// filter only OnHold and Approved state
/// 001001 -> 9
/// so you can pass 9 to the filter to get only OnHold and Approved
/// </summary>
[JsonConverter(typeof(JsonStringEnumConverter))]
[Flags]
public enum State

Choose a reason for hiding this comment

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

Un manière plus simple de déclarer un enum [Flags] serait d'utiliser l'opérateur Bit Shift, par ex. 1<<1 pour 2, 1<<2 pour 4, 1<<3 pour 8, etc. Source : https://aaronbos.dev/posts/csharp-flags-enum

/// any types, it is not possible to fetch draft events from there. This controller is used to only
/// create/update draft events.
///
/// Under the hood, this controller uses the IDraftEventService to manage the data by passing the

Choose a reason for hiding this comment

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

Bonne idée de documenter à la fois la fonctionnalité d'un point de vue utilisateur et d'un point de vue interne au code

/// consume and the open data API accessible from the front-end or external parties that want to
/// show data.
///
/// Under the hood, this controller uses the IEventService to manage the event data fetching, as well as

Choose a reason for hiding this comment

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

Idem que le commentaire précédent

/// without having to know your own id. This is useful for updating your own information or fetching
/// your profile information. This can work for either a moderator or an organizer.
///
/// Under the hood, this controller uses the IUserService to manage the data by passing the userId found in

Choose a reason for hiding this comment

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

Idem que le commentaire précédent

/// setup of the application by allowing having a public endpoint to create a starting moderator.
/// Without the header x-api-key, this function will not worked.
///
/// Also, the configuration of the access key is done in the .env file. If you want to remove this functionnality.

Choose a reason for hiding this comment

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

Très bien fait d'avoir documenté les requis de cette fonctionnalité

/// A moderator can create, get, toggle the active state of an organizer and get a list of all organizers.
///
/// Under the hood, this controller uses the IUserService to manage the data. It also uses the IAuthService to
/// create a new user in the Supabase database. Finally, it uses the IEmailService to send an email to the newly

Choose a reason for hiding this comment

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

Il faudrait s'assurer d'avoir les bons réglages dans le fichier .env pour que l'envoi de courriels fonctionne dans Supabase

Copy link

@MysticFragilist MysticFragilist merged commit 633c467 into main Jun 26, 2024
3 checks passed
@MysticFragilist MysticFragilist deleted the ftr/documentation-classes-methods branch June 26, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants