Skip to content
This repository has been archived by the owner on Sep 3, 2023. It is now read-only.

refactor: extract table logic #433

Merged
merged 2 commits into from
Sep 18, 2022
Merged

Conversation

FrediKats
Copy link
Member

ref to #432

Попытался начать писать тесты на очередь и наткнулся на то, что логика очереди в GoogleIntegration лежит частично. Постарался минимальными изменениями вынести первый кусок - воркер.

  • Что хочется в итоге: возможность написать текст, когда меняется сабмишен и мы можем проверить, что в очередь попал запрос на перегенерацию
  • Что нужно следующим шагом сделать: резать GoogleTableAccessor, вынести с него CreateSpreadsheetAsync в апликейшен

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 6 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@Mihinator3000 Mihinator3000 left a comment

Choose a reason for hiding this comment

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

Немного смущает название TableManagement, а так вроде окей
Ну и sonar там ненужные юзинги нашел

Docker/build.dockerfile Show resolved Hide resolved
@FrediKats FrediKats merged commit 38f0ca4 into master Sep 18, 2022
@FrediKats FrediKats deleted the refactor/extract-table-logic branch September 18, 2022 05:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants