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

Approved by
Danis Kirasirov Anton Sattarov
Ready to merge by members who can write to the target branch.
  • 1 commit and 1 merge commit will be added to refactor/PRXS-3053-Files (squashes 6 commits).
  • Source branch will be deleted.

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 - твое мнение нужно?

  • Semyon Krestyaninov
    Semyon Krestyaninov @krestyaninov
    Author Developer

    @softzilla Мы с Антоном обсуждаем, как лучше реализовать передачу параметров в proto, нужно твое мнение.

    Первый вариант - передавать конкретные поля (например, только size и url). Плюс в том, что сразу видно, какие поля можно изменить, и не придется гадать, что именно обновится.

    Второй вариант - передавать весь объект File. Так мы делаем везде, и если структура файла поменяется, не придется переписывать метод. Но минус в том, что не всегда понятно, какие поля можно обновить.

  • Please register or sign in to reply
  • Anton Sattarov approved this merge request

    approved this merge request

  • Anton Sattarov requested review from @softzilla and removed review request for @sattarov

    requested review from @softzilla and removed review request for @sattarov

    • Please register or sign in to reply