#577 Расширить фабрику клиентов Persistance #8

Merged
on.nemtina merged 11 commits from ClientFactory into master 2024-12-13 09:43:40 +05:00
Collaborator
No description provided.
rs.efremov added 1 commit 2024-12-09 14:46:59 +05:00
rs.efremov added 1 commit 2024-12-09 17:11:47 +05:00
ng.frolov requested review from on.nemtina 2024-12-09 17:14:37 +05:00
aa.shibalkin requested changes 2024-12-10 10:41:24 +05:00
aa.shibalkin left a comment
Collaborator

Требуется исправить некоторые типы логгера + обращаться к словарю через TryGetValue.
Также в event сервисе я использую Guid системы вместо string

Требуется исправить некоторые типы логгера + обращаться к словарю через TryGetValue. Также в event сервисе я использую Guid системы вместо string
@ -0,0 +3,4 @@
using Refit;
namespace Persistence.Client.Clients.Base;
public class BaseClient
Collaborator

Вероятно, это абстрактный класс

Вероятно, это абстрактный класс
@ -0,0 +7,4 @@
{
private readonly ILogger logger;
public BaseClient(ILogger logger)
Collaborator

Логгер должен быть Ilogger от типа класса

Логгер должен быть Ilogger от типа класса
@ -0,0 +46,4 @@
/// <param name="categoryId"></param>
/// <param name="token"></param>
/// <returns></returns>
Task<IEnumerable<MessagesStatisticDto>> GetStatistics(string autoDrillingSystem, int categoryId, CancellationToken token);
Collaborator

У меня используется не string, а Guid (SystemId)

У меня используется не string, а Guid (SystemId)
@ -0,0 +10,4 @@
{
private readonly IRefitSetpointClient refitSetpointClient;
public SetpointClient(IRefitSetpointClient refitSetpointClient, ILogger logger) : base(logger)
Collaborator

Ilogger от SetpointClient можно подставить

Ilogger от SetpointClient можно подставить
@ -0,0 +25,4 @@
public static Exception GetPersistenceException(this IApiResponse response)
{
var exception = ExceptionsDictionary
Collaborator

Обращайся к словарю через TryGetValue.

Обращайся к словарю через TryGetValue.
@ -30,0 +61,4 @@
public ITechMessagesClient GetTechMessagesClient()
{
var restClient = RestService.For<IRefitTechMessagesClient>(httpClient, RefitSettings);
var client = new TechMessagesClient(restClient, logger);
Collaborator

Можно через IServiceProvider доставать нужный логгер и инициализировать им клиента

Можно через IServiceProvider доставать нужный логгер и инициализировать им клиента
rs.efremov added 2 commits 2024-12-10 13:55:05 +05:00
rs.efremov added 1 commit 2024-12-10 13:55:38 +05:00
rs.efremov added 11 commits 2024-12-11 11:29:56 +05:00
rs.efremov added 2 commits 2024-12-11 14:56:33 +05:00
rs.efremov added 1 commit 2024-12-12 11:56:25 +05:00
rs.efremov added 1 commit 2024-12-13 09:39:37 +05:00
on.nemtina merged commit 3df736fdc9 into master 2024-12-13 09:43:40 +05:00
on.nemtina deleted branch ClientFactory 2024-12-13 09:43:40 +05:00
Sign in to join this conversation.
No reviewers
No Label
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: on.nemtina/persistence#8
No description provided.