Skip to content

Files: ошибка учета удаленных файлов при подсчете в FilesCountLimiter#656

Open
semrosin wants to merge 76 commits intomasterfrom
files-to-solutions
Open

Files: ошибка учета удаленных файлов при подсчете в FilesCountLimiter#656
semrosin wants to merge 76 commits intomasterfrom
files-to-solutions

Conversation

@semrosin
Copy link
Contributor

@semrosin semrosin commented Mar 5, 2026

No description provided.

@semrosin semrosin requested a review from DedSec256 March 5, 2026 17:51
@semrosin semrosin added the bug Something isn't working label Mar 5, 2026
if (!existingStatuses.Succeeded) return false;

var existingIds = existingStatuses.Value.Select(f => f.Id).ToList();
var existingIds = existingStatuses.Value.Where(f => f.Status == "ReadyToUse").Select(f => f.Id).ToList();
Copy link
Contributor

Choose a reason for hiding this comment

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

А почему не учитываются файлы в процессе загрузки или файлы, которые не удалось удалить?

Copy link
Contributor Author

@semrosin semrosin Mar 9, 2026

Choose a reason for hiding this comment

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

Жалко пользователя, который не может загрузить файлы из-за того что у нас не получилось прошлые удалить(((

Copy link
Contributor

Choose a reason for hiding this comment

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

Тогда, наверное, стоит разобраться, почему не получилось их удалить (или почему не отобразилось, что они в процессе удаления). Данное исправление выглядит некорректным

@semrosin semrosin force-pushed the files-to-solutions branch from a8e4f8c to 47c6e8f Compare March 9, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants