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.

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

Сортировка заданий по дате публикации #518

Merged
merged 4 commits into from
Mar 18, 2025

Conversation

YuriUfimtsev
Copy link
Contributor

@YuriUfimtsev YuriUfimtsev commented Mar 12, 2025

Реализовано в соответствии с #503 (и с учетом ограничений Entity Framework Core 2.x).
image

image

Реализовано отображение по убыванию даты публикации: самое верхнее задание в режиме редактирования имеет самый поздний срок публикации.
В таблице задания располагаются слева направо по убыванию даты публикации.

Comment on lines 52 to 60

// todo: перенести сортировку домашних работ на уровень репозитория после обновления до EF Core 5.x
courses = courses.Select(c =>
{
c.Homeworks = c.Homeworks
.OrderBy(h => h.PublicationDate)
.ToList();
return c;
}).ToArray();
Copy link
Contributor

Choose a reason for hiding this comment

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

здесь можно ничего не сортировать

Comment on lines 200 to 209
// todo: перенести сортировку домашних работ на уровень репозитория после обновления до EF Core 5.x
coursesWithValues = coursesWithValues
.Select(c =>
{
c.Homeworks = c.Homeworks
.OrderBy(h => h.PublicationDate)
.ToList();
return c;
}).ToArray();

Copy link
Contributor

Choose a reason for hiding this comment

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

и здесь

@DedSec256 DedSec256 merged commit 4791793 into master Mar 18, 2025
1 check passed
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