Skip to content

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

Closed
semrosin wants to merge 76 commits into
masterfrom
files-to-solutions
Closed

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

Conversation

@semrosin

@semrosin semrosin commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

@semrosin semrosin Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
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
@DedSec256 DedSec256 closed this Apr 25, 2026
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