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

Tutorial #94

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Tutorial #94

wants to merge 18 commits into from

Conversation

kalenav
Copy link
Contributor

@kalenav kalenav commented May 6, 2023

Respective KB pull request: ostis-ai/ims.ostis.kb#20

100% {
outline-color: rgba(255, 0, 0, 0);
}
}
Copy link
Member

Choose a reason for hiding this comment

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

please add new line

text: 'Добро пожаловать в обучение! Здесь Вы узнаете об основном функционале системы. Обучение всегда можно завершить досрочно нажатием кнопки "Abort tutorial" в правом нижнем углу этого окна. Ваше первое задание - поиск сущности в базе знаний по идентификатору. Нажмите на поле поиска.'
},
eng: {
title: '',
Copy link
Member

Choose a reason for hiding this comment

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

please add english description

@@ -0,0 +1,327 @@
SCWeb.ui.Tutorial = {
Copy link
Member

Choose a reason for hiding this comment

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

maybe change tuttorial to tutorial mode?

break;
}
}
}
Copy link
Member

Choose a reason for hiding this comment

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

please add new line


addStageCompletionListener: function(stageNumber) {
switch(stageNumber) {
case 0:
Copy link
Member

Choose a reason for hiding this comment

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

change numbers to logical names

}, 200)
break;
case 7:
this.clickListener();
Copy link
Member

Choose a reason for hiding this comment

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

why in this case no break?

desc: {
rus: {
title: 'Поиск по основному идентификатору',
text: 'Это - поле поиска сущности по идентификатору. В него можно вводить как стандартные, так и системные идентификаторы. Давайте найдём сущность с основным идентификатором "транзитивное отношение".'
Copy link
Member

Choose a reason for hiding this comment

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

maybe its not a good idea to save all this text here, maybe we can do it through knowledge base?

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

2 participants