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

Шаги для перемещения по ячейкам табличного документа #2004

Merged
merged 4 commits into from
Jul 16, 2023

Conversation

1cgh
Copy link
Contributor

@1cgh 1cgh commented Jul 3, 2023

сделанные изменения

  • добавлены шаги для перемещения по ячейкам табличного документа относительно текущей ячейки

@VACIBot
Copy link
Collaborator

VACIBot commented Jul 3, 2023

Нужно запустить тесты?

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 3, 2023

@1cgh
Привет! Ты добавил экспортный сценарий.
Но тут есть проблема, его некуда помещать при сборке VASingle. Там ведь пользователь получает только один epf файл.
Поэтому нужно шаги для VA реализовывать кодом.

@1cgh
Copy link
Contributor Author

1cgh commented Jul 3, 2023

@Pr-Mex яснопонятно. есть пример, как это проще сделать для таких случаев? "такие" - это когда в новом шаге используются только другие шаги, своего кода по сути нет.
тупо через Ванесса.Шаг() в коде реализации шагов?

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 3, 2023

@Pr-Mex яснопонятно. есть пример, как это проще сделать для таких случаев? "такие" - это когда в новом шаге используются только другие шаги, своего кода по сути нет. тупо через Ванесса.Шаг() в коде реализации шагов?

@1cgh
Да, Ванесса.Шаг() можно использовать в коде шага, если этот шаг обычный, не асинхронный.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 4, 2023

@1cgh
Ты будешь переделывать этот реквест? Или его лучше закрыть?

@1cgh
Copy link
Contributor Author

1cgh commented Jul 11, 2023

@Pr-Mex доделаю)

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 11, 2023

@1cgh
Ок, а то скоро релиз )

@1cgh
Copy link
Contributor Author

1cgh commented Jul 12, 2023

@Pr-Mex фичу убрал, шаги сделал кодом, поместил в работу с интерфейсом

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 12, 2023

@1cgh
Создай базу в режиме совместимости 8.2 и открой epf, где менялся код.
Будут ошибки. Их надо исправить.
Например я увидел использование СтрНайти

см тут
https://github.com/Pr-Mex/vanessa-automation/blob/develop/docs/FAQ/%D0%A0%D0%B5%D0%B6%D0%B8%D0%BC%20%D1%81%D0%BE%D0%B2%D0%BC%D0%B5%D1%81%D1%82%D0%B8%D0%BC%D0%BE%D1%81%D1%82%D0%B8%208.2.md

@1cgh
Copy link
Contributor Author

1cgh commented Jul 12, 2023

@Pr-Mex поправил

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 12, 2023

run tests

2 similar comments
@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 13, 2023

run tests

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 14, 2023

run tests

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jul 16, 2023

@1cgh
Спасибо!

@Pr-Mex Pr-Mex merged commit 91320b1 into Pr-Mex:develop Jul 16, 2023
1 check passed
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.

None yet

3 participants