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

Java Servlet Todo List - local deployment of todo application #776

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DKARAGODIN
Copy link
Contributor

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru

Description of changes:

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

Мержить одновременно с yandex-cloud-examples/yc-serverless-servlet#1

@yfm-team
Copy link

yfm-team commented Jun 9, 2024

Все готово!
Commit: b216551
Проверьте результат: RU.
Check the result: EN.

@myusosnovskay myusosnovskay added the cloud-functions Cloud Functions label Jun 10, 2024
@myusosnovskay
Copy link
Contributor

Разработчики сервиса оставили комментарии к коду: yandex-cloud-examples/yc-serverless-servlet#1 (comment).

@DKARAGODIN
Copy link
Contributor Author

DKARAGODIN commented Oct 4, 2024

Исправил с учетом комментария

@myusosnovskay
Copy link
Contributor

@DKARAGODIN Здравствуйте! Попробовали запустить сценарий с учетом ваших правок, и он не заработал. Подскажите, пожалуйста, работает ли он у вас?

Исходный сценарий по-прежнему рабочий.

@DKARAGODIN
Copy link
Contributor Author

У вас не работает локальная версия на Tomcat или в облаке развернутая?

Когда месяц назад я обновлял коммит, я задеплоил в облаке новую версию. Все работает и до сих пор. Сейчас убедился, что да, новая версия работает.
Вот она: https://functions.yandexcloud.net/d4eeafc2opphkumo8k38

@myusosnovskay
Copy link
Contributor

@DKARAGODIN Все ли правки, которые необходимы для того, чтобы запустить обновленный код, вы учли в документации? Возможно, нужно запускать функцию с другими параметрами?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud-functions Cloud Functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants