Skip to content

Feature/assignee#19

Merged
Leenday merged 6 commits intodevelopfrom
feature/assignee
Dec 26, 2022
Merged

Feature/assignee#19
Leenday merged 6 commits intodevelopfrom
feature/assignee

Conversation

@Leenday
Copy link
Copy Markdown
Owner

@Leenday Leenday commented Dec 20, 2022

No description provided.

@Leenday
Copy link
Copy Markdown
Owner Author

Leenday commented Dec 20, 2022

В этой ветке присутствуют изменения из feature/tasks, т.к. последняя не влита, на момент формирования ПР, а изменения из нее необходимы для дальнейшей разработки.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 20, 2022

Uffizzi Preview deployment-9364 was deleted.

Copy link
Copy Markdown

@greybutton greybutton left a comment

Choose a reason for hiding this comment

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

реквест содержит код от другой задачи

Copy link
Copy Markdown

@greybutton greybutton left a comment

Choose a reason for hiding this comment

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

презентеры нужно использовать во всем проекте

);
}

// Task.propTypes = TaskPresenter.shape();
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

если код не нужен, то он удаляется, а не комментируется

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Удалил, спасибо

// Task.propTypes = TaskPresenter.shape();
Task.propTypes = {
task: PropTypes.shape().isRequired,
onClick: PropTypes.func.isRequired,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

почему удалил onClick?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Из-за isRequired была ошибка о том, что функция - undefined. Сейчас вернул без нее, ошибка ушла.


def task_params
params.require(:task).permit(:name, :description, :author_id, :assignee_id, :state_event, :expired_at)
params.require(:task).permit(:id, :name, :description, :author_id, :assignee_id, :state_event, :expired_at)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

почему ид добавил?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Из-за ошибки при обновлении таски:

web_1  | Started PUT "/api/v1/tasks/60" for 172.30.0.1 at 2022-12-23 07:28:56 +0000
web_1  | Cannot render console from 172.30.0.1! Allowed networks: 127.0.0.0/127.255.255.255, ::1
web_1  | Processing by Api::V1::TasksController#update as JSON
web_1  |   Parameters: {"task"=>{"id"=>60, "name"=>"fdsaf", "description"=>"asdfa", "assignee_id"=>4, "author_id"=>2}, "id"=>"60"}
web_1  |   Task Load (0.7ms)  SELECT "tasks".* FROM "tasks" WHERE "tasks"."id" = $1 LIMIT $2  [["id", 60], ["LIMIT", 1]]
web_1  |   ↳ app/controllers/api/v1/tasks_controller.rb:27:in `update'
web_1  | Unpermitted parameter: :id

@Leenday Leenday requested a review from greybutton December 23, 2022 07:36
Copy link
Copy Markdown

@greybutton greybutton left a comment

Choose a reason for hiding this comment

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

презентеры это про сущности в проекте, а не про пропсы компонентов. К юзерам и таскам во всем проекте применить презентеры, пропсы вернуть как было

@Leenday Leenday merged commit f3e2d5e into develop Dec 26, 2022
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