Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add documentation to several schedule endpoints #836

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Add documentation to several schedule endpoints #836

wants to merge 2 commits into from

Conversation

Soctarian
Copy link

No description provided.

@Soctarian Soctarian added the documentation Improvements or additions to documentation label Apr 3, 2022
@Soctarian Soctarian self-assigned this Apr 3, 2022
@Soctarian Soctarian linked an issue Apr 3, 2022 that may be closed by this pull request
Add documentation for endpoints in SchedulesController (swagger) #827
Open
@Soctarian Soctarian requested a review from Ssashaa April 6, 2022 07:16
Ssashaa
Ssashaa reviewed Apr 11, 2022
View reviewed changes
CharlieBackend.Api/Controllers/SchedulesController.cs
/// </remarks>
///
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove empty space

CharlieBackend.Api/Controllers/SchedulesController.cs
/// studentGroupID: student group id to wich is event occurrences attached to;
/// themID, mentorID: them and mentor ids
/// </remarks>
///
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove empty space

CharlieBackend.Api/Controllers/SchedulesController.cs
@@ -85,6 +114,15 @@ public async Task<ActionResult<DetailedEventOccurrenceDTO>> GetDetailedEventOccu
/// <summary>
/// Returns the list of events depending on the filtering rules set
/// </summary>
///
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove empty space

CharlieBackend.Api/Controllers/SchedulesController.cs
/// <remarks>
/// Return result by filtering data with parameters
/// </remarks>
///
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove empty space

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ssashaa Ssashaa Ssashaa left review comments

@DmitryTkachenko DmitryTkachenko DmitryTkachenko approved these changes

@BobTheSuperman BobTheSuperman BobTheSuperman approved these changes

@andy-dendy-repo andy-dendy-repo andy-dendy-repo approved these changes

@apyura apyura Awaiting requested review from apyura

@AccorDDD AccorDDD Awaiting requested review from AccorDDD

@ArsenVlasov ArsenVlasov Awaiting requested review from ArsenVlasov

@ArsenMaistryshyn ArsenMaistryshyn Awaiting requested review from ArsenMaistryshyn

Assignees

@Soctarian Soctarian

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add documentation for endpoints in SchedulesController (swagger)
5 participants
@Soctarian @DmitryTkachenko @BobTheSuperman @andy-dendy-repo @Ssashaa