Skip to content
Snippets Groups Projects

В контракт сервиса файлов добавлен метод обновления информации о файле

Open В контракт сервиса файлов добавлен метод обновления информации о файле
refactor/PRXS-3053-RefactorFiles into refactor/PRXS-3053-Files
1 unresolved thread
Open Semyon Krestyaninov requested to merge refactor/PRXS-3053-RefactorFiles into refactor/PRXS-3053-Files
1 unresolved thread

Описание

Issue: perxis#3053

  • Добавлен метод UpdateFile

Требуется для MR:

Edited by Semyon Krestyaninov

Merge request reports

Ready to merge by members who can write to the target branch.
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
proto/files/files.proto
56 56 }
57 57 message AbortUploadResponse {}
58 58
59 message MoveUploadRequest {
60 MultipartUpload upload = 1;
59 message UpdateFileRequest {
60 string file_id = 1;
  • Anton Sattarov
    Anton Sattarov @sattarov ·
    Maintainer

    @krestyaninov сейчас подумал а почему сразу не File?

  • Semyon Krestyaninov
    Semyon Krestyaninov @krestyaninov ·
    Author Developer

    @sattarov Мне кажется, что не лучшая идея передавать всю структуру вместо конкретных полей. Например, если сразу передавать файл, то при отправке запроса мы сможем указать size и url, но они не обновятся. Думаю это не очень дружелюбно. А так сразу видно, какие поля можно изменить.

  • Anton Sattarov
    Anton Sattarov @sattarov ·
    Maintainer

    а мне кажется наоборот - изменится структура файла и метод придется api метода менять.

    мы во всем api так делаем - принимаем обьект и тоже не все поля при обновлении меняем - а тут идем по другому пути

    Edited by Anton Sattarov
  • Anton Sattarov
    Anton Sattarov @sattarov ·
    Maintainer

    @softzilla - твое мнение нужно?

  • Please register or sign in to reply
Please register or sign in to reply