Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Docs #18

wants to merge 3 commits into from

Conversation

strug
Copy link
Contributor

@strug strug commented Mar 24, 2022

No description provided.

@strug strug changed the title Ft/docs Docs Mar 24, 2022
@@ -4,11 +4,15 @@
* [Процесс с полным прохождением 3дс версии 1.0 и 2.0](./flows/full_three_ds_v1_v2_flow_steps.md)
* [Простой процесс с редиректом и поллингом статусов после всех операций](./flows/simple_redirect_with_polling_flow_steps.md)

2. Если подходящий вам процесс найден, процесс продолжается по следующему пути:
2. Если подходящий вам процесс найден, необходимо выполнить следующие шаги:
Copy link
Contributor

Choose a reason for hiding this comment

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

...найден, то далее необходимо выполнить следующие шаги:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

...найден, то далее необходимо выполнить следующие шаги:

не мне как раз такие паразиты не нравятся в "то далее" не несет смысловой нагрузки

Copy link
Contributor

Choose a reason for hiding this comment

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

Ты используешь конструкцию "Если-то", но выкинув "то", оно кажется незаконченным.
Слова-связки норм в языке)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ты используешь конструкцию "Если-то", но выкинув "то", оно кажется незаконченным. Слова-связки норм в языке)

ну в том то и дело, что умным людям не надо писать если - то , в контексте все очевидно, а буков меньше (пиши сокращай)

Copy link
Contributor

Choose a reason for hiding this comment

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

ля кучеряво ты расписал под шконарь конструкции русского языка. Сразу видно ровный пасан, а на фраер залетный. Такому и аппрув поставить не западло)

docs/flow_using_lib.md Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Mar 24, 2022

Codecov Report

Merging #18 (c427240) into master (4a3cddc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #18   +/-   ##
=========================================
  Coverage     65.02%   65.02%           
  Complexity      357      357           
=========================================
  Files            99       99           
  Lines          1135     1135           
  Branches         97       97           
=========================================
  Hits            738      738           
  Misses          327      327           
  Partials         70       70           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a3cddc...c427240. Read the comment docs.

@@ -1,4 +1,8 @@
# adapter-flow-lib

Данная библиотека предназначена для упрощенного процесса реализации взаимодействия с провайдерами.

Choose a reason for hiding this comment

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

... для упрощения реализации процесса взаимодействия с провайдерами

Copy link
Contributor

@D-Baykov D-Baykov left a comment

Choose a reason for hiding this comment

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

коти бродяга

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.

4 participants