on.nemtina/persistence
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix/mock-methods-for-change-log #27

Merged
on.nemtina merged 7 commits from fix/mock-methods-for-change-log into master 2025-02-06 17:23:32 +05:00
Conversation 0 Commits 7 Files Changed 5 +179 -1
on.nemtina commented 2025-02-06 16:15:30 +05:00
Owner
Copy Link

Мок-данные для того, чтобы фронты начали разработку

Мок-данные для того, чтобы фронты начали разработку
on.nemtina added 2 commits 2025-02-06 16:15:30 +05:00
on.nemtina requested review from ng.frolov 2025-02-06 16:15:37 +05:00
ng.frolov was assigned by on.nemtina 2025-02-06 16:15:39 +05:00
on.nemtina changed title from fix/mock-methods-for-change-log to WIP: fix/mock-methods-for-change-log 2025-02-06 16:15:40 +05:00
on.nemtina added 1 commit 2025-02-06 16:15:44 +05:00
on.nemtina added 2 commits 2025-02-06 16:57:45 +05:00
on.nemtina changed title from WIP: fix/mock-methods-for-change-log to fix/mock-methods-for-change-log 2025-02-06 16:57:59 +05:00
on.nemtina added 1 commit 2025-02-06 17:07:13 +05:00
ng.frolov requested changes 2025-02-06 17:12:00 +05:00
DD.Persistence.API/Controllers/ChangeLogController.cs
@ -177,0 +205,4 @@
public async Task<ActionResult<IEnumerable<HistoryChangeLogDto>>> HistoryChangeLogAsync([FromQuery] ChangeLogRequest request, CancellationToken token)
{
var userId = Guid.CreateVersion7();
var changeLogItemCurrentId = Guid.CreateVersion7();
ng.frolov commented 2025-02-06 17:08:54 +05:00
Collaborator
Copy Link

👍

👍
DD.Persistence.API/Controllers/ChangeLogController.cs
@ -177,0 +243,4 @@
var result = new List<HistoryChangeLogDto>() {
new() {
Comment = "Петров И. Ю. попросил внести изменения",
DateTime = DateTimeOffset.UtcNow,
ng.frolov commented 2025-02-06 17:11:54 +05:00
Collaborator
Copy Link

Дата тут и дата в 214 строке будут отличаться, это приведет к плавающему багу из-за округления.
Просто сделай локальную переменную с датой в начале метода и используй её

Дата тут и дата в 214 строке будут отличаться, это приведет к плавающему багу из-за округления. Просто сделай локальную переменную с датой в начале метода и используй её
on.nemtina added 1 commit 2025-02-06 17:21:36 +05:00
on.nemtina merged commit 43289d939b into master 2025-02-06 17:23:32 +05:00
on.nemtina deleted branch fix/mock-methods-for-change-log 2025-02-06 17:23:32 +05:00
Sign in to join this conversation.
Reviewers
No reviewers
ng.frolov
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
ng.frolov
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: on.nemtina/persistence#27
No description provided.
Delete Branch "fix/mock-methods-for-change-log"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?