Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

Invites #42

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

Invites #42

wants to merge 23 commits into from

Conversation

pashakbit
Copy link
Member

@pashakbit pashakbit commented Apr 19, 2019

Так как инвайты пилились на основе ветки с правами, а права еще не зарелижены, то смореть только последний коммит, все коммиты переребейзил туда, чтобы было проще смотреть.

iconClasses() {
return [
'simple-preloader',
'_' + this.color,
Copy link
Contributor

Choose a reason for hiding this comment

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

es6 забанили? )

`_${this.color}`

Copy link
Member Author

@pashakbit pashakbit Apr 19, 2019

Choose a reason for hiding this comment

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

Да, я что-то аналогичный скопипизжил и не переделал почему-то) Поправлю, спасибо)

Copy link
Member Author

Choose a reason for hiding this comment

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

Поправил

hasAccessToModule: (
{ permissions, nextRoute },
{ currentVendorId }
) => (resourceId: string = '') => {
Copy link
Contributor

Choose a reason for hiding this comment

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

А где-нибудь этот resourceId в итоге передаётся? )

Copy link
Member Author

Choose a reason for hiding this comment

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

Он обязательно будет передаваться в Games, так как права будут по id'шнику игры, просто в rbac'е не стал все модули переводить, отдельная ветка у меня локально лежит, в ней я это сделал уже

Copy link
Member Author

Choose a reason for hiding this comment

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

Отдельно уже поправил, чтобы быстрее выпустить

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants