sinfin / folio

MIT License
6 stars 2 forks source link

WIP: Rozpuštění Folio::Account ve Folio::User [GEN-55] #186

Closed foton closed 6 months ago

foton commented 7 months ago

Touches #180 , https://github.com/sinfin/foliolized_auctify/issues/90

Kompletní odstranění Folio::Accounts. V rámci migrace jsou stávající accounts přesunuta do Folio::Users. Těm přibyl atribut user.superadmin? ( = account.has_role?("superuser")). Ostatní account role jsou nahrány jako site_roles (skrz user.site_user_links) pro všechny existující Folio::Site Přihlašování do console se tedy děje skrz user/sign_in.

Všechny autorizace se dějí skrz skrz metodu can_now?(:akce, objekt) (pro kontrolery) nebo user.can_now?(:akce, objekt). Nastavení oprávnění se děje skrzFolio::Ability, které pro správnou funkci potřebujesite( v kontrollerech si vezme current_site; při volání skrz model ji očekává vobjekt.site`.

Není dotaženo:

foton commented 6 months ago

@mreq Potřeboval bych nějak vyřešit zobrazování "impersonating_header" ve frontendu (Dummy app). Funguje mi to v konzoli, ale ta samá buňka se nedá použít ve frontendu (nezná to pak stop_impersonation_console_users_url). A taky se mi to nepodařilo nastylovat (ideálně horný proh přes celou stránku). image

foton commented 6 months ago

JInak je to ready na review a merge do v5

foton commented 6 months ago

Jo a možná b to chtělo upravit "role management". Teď je to takhle . image

Zobrazeny všechny site ke kterým má uživatel admin přístup. Zatržítko u domény přidává/maže záznam site_user_link a zatržítka u rolí přidávají/ubírají role.

notion-workspace[bot] commented 6 months ago

Merge accounts & users