Skip to content

feature: simplify user creation (https://webitel.atlassian.net/browse/WTEL-9735)[WTEL-9735]#1323

Open
RomanZaritskyi wants to merge 7 commits into
mainfrom
feature/WTEL-9735/simple-user-creation
Open

feature: simplify user creation (https://webitel.atlassian.net/browse/WTEL-9735)[WTEL-9735]#1323
RomanZaritskyi wants to merge 7 commits into
mainfrom
feature/WTEL-9735/simple-user-creation

Conversation

@RomanZaritskyi

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/app/locale/en/en.js Outdated
chatName: 'Chat display name',
temporaryPassword: 'Temporary password',
extensionsHelperText: 'Extension must contain only digits',
// WTEL-9735 Підзадача 6

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

коментарі би англом пасувало би писати)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

та його видалити треба..))


import { useUserAccessControl } from '../../../../../app/composables/useUserAccessControl';
import openedTabComponentMixin from '../../../../../app/mixins/objectPagesMixins/openedObjectTabMixin/openedTabComponentMixin';
import ContactsAPI from '../api/contacts.js';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

по можливості краще імпортувати одразу з @webitel/api-services/api

lang="scss"
scoped
>
@use '@webitel/ui-sdk/src/css/main' as *;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

а мейн точно потрібен?

Comment on lines +183 to +185
import RolesAPI from '../../../../permissions/modules/roles/api/roles';
import DevicesAPI from '../../devices/api/devices';
import LicenseAPI from '../../license/api/license';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

якщо можна імпортувати з @webitel/api-services/api, то було би круто)
але якщо ні то ні

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

з того що бачу то поки не можна

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ці апішки унікальні для адмінки. Пропоную їх поки не виности в api-services. Але якщо треба і на це є причини то можу винести.
Я просто думаю що яещо вони ніде не потрібні, то їх винесення в api-services — це + до складності розробки в подальному.

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.

2 participants