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

fix: grantee select values filtered [WTEL-4001] #32

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

Conversation

VladimirBeria
Copy link
Contributor

No description provided.

@dlohvinov
Copy link
Collaborator

охх, Вов. Хотілося би мені тебе похвалити, але в цьому пулл реквесті нема за шо
себе доречі теж

в чому проблема: є пермішенс компоненти: в них передбачена реюзабельність через певну абстрацію -- неймспейс пропсом. отже, незалежно від контексту, можна просто передати неймспейс, і все полетить. це тягне на модуль одного рівня з контактами, а не його підлеглий.
ось тут перша помилка пулл реквесту: контакт ід взятий прямо по захардкодженому неймспейсу, у попапі. я би рекомендував його взяти у the-permissions.vue, по динамічному неймспейсу, і передати у попап пропсом, а з попапу у селект

йдемо далі. у нас є стор пермішенів. він трохи кривуватий: логіка у ньому абстрактна -- бери, копіюй і тули де хочеш, а от використання у нього пряме -- логіка створюється не через якусь "факторі", а просто тупо написано і передано у контакти. тобто, код реюзабельний, а використання -- ні. що треба зробити? нууу, варіанти 2: забити болт (1), або покласти його разом з компонентами на один рівень з контактами, та переробити з використанням BaseStoreModule.js з вебітел-юі/src/store (2). далі створити у підлеглому контактам модулі пермішенів стор, який би використовував ту "факторі" реюзабельного модуля, який лежить у модулях на одному рівні з контактами.

далі апі. тут теж хочеться зробити якусь функцію, яка би отримувала бейс юрл, а вертала би { getList, patch } і все шо треба взагалі: як для пермішенс апі, так і для ролс апі. відповідно, "факторі" апішек мають лежати у модулі пермішенів на одному рівні з контактами, а їх використання -- у пермішенах, підлеглих контактам.

тобто, має бути

src/modules/permissions, у якому будуть компоненти пермішенів, і факторі для стора та апішок, і src/modules/contacts/modules/permissions, у якому будуть створені стор і апішки з тих факторей.

@dlohvinov
Copy link
Collaborator

поки писав, подумав що треба буде якось прокинути ті створені апішки на компонент. як? -- тут очевидного рішення немає, тому захочую проявити фантазію. можна або у компонент пропсом, або у стор. я би це напевне через стор мутив, щоб можна було достукатись по неймспейсу і не видумувати пропс апі метод.

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