Added pages #8

Open
jezv wants to merge 1 commit from web into interfaces
Owner

изображение
изображение

![изображение](/attachments/ae8e0050-1242-4e52-b9ee-4018264188f2) ![изображение](/attachments/52c3129a-0f49-45c5-94b8-0bc237fd7640)
VolovikovAlexander left a comment

Нужны доработки

Нужны доработки
@ -0,0 +5,4 @@
namespace PersonalAccount.Api.Web.Controllers;
[ApiController]
First-time contributor

Оформление кода?

Оформление кода?
@ -0,0 +28,4 @@
return corporation;
}
[HttpPut("{id}/settings")]
First-time contributor

У Вас на форме

<form method="post">

Тут будет URL POST Index вместо

[HttpPut("{id}/settings")]

Проверьте пжл

У Вас на форме ` <form method="post">` Тут будет URL POST Index вместо ` [HttpPut("{id}/settings")]` Проверьте пжл
@ -0,0 +31,4 @@
[HttpPut("{id}/settings")]
public async Task<IActionResult> UpdateSettings(Guid id, Settings settings)
{
var corporation = await context.Copropations.FindAsync(id);
First-time contributor

А остальные параметры. Например - наименование филиала?

А остальные параметры. Например - наименование филиала?
@ -0,0 +37,4 @@
return NotFound();
}
corporation.Settings = settings;
First-time contributor

Нет проверки.


5. Обазательно сделать проверку валидации данных.
https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/Src/PersonalAccount.Domain/Models/DomainModel.cs
см Validate
Нет проверки. ``` 5. Обазательно сделать проверку валидации данных. https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/Src/PersonalAccount.Domain/Models/DomainModel.cs см Validate ```
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin web:web
git switch web

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch interfaces
git merge --no-ff web
git switch web
git rebase interfaces
git switch interfaces
git merge --ff-only web
git switch web
git rebase interfaces
git switch interfaces
git merge --no-ff web
git switch interfaces
git merge --squash web
git switch interfaces
git merge --ff-only web
git switch interfaces
git merge web
git push origin interfaces
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jezv/VolovikovASPNET!8
No description provided.