A simple module that provide functionality to add different path to brand according to roles.On click of brand user will redirect
to particular path as per role.Admin can configure visibility of site name,site logo and site slogan.

Dependencies

This module requires no modules outside of Drupal core.

Project link

https://www.drupal.org/project/site_branding_per_role

Git instructions

git clone --branch 8.x-1.x https://git.drupalcode.org/project/site_branding_per_role.git

PAReview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-site_branding_...

Comments

bsumit5577 created an issue. See original summary.

sumit-k’s picture

Issue summary: View changes
shaktik’s picture

Issue summary: View changes

git clone at 1 line with URL.

rksyravi’s picture

Priority: Normal » Critical
Status: Active » Needs work

Hi @bsumit5577,

Thank you for contribution!!!

As I can see your git branching is not as per the drupal guidelines. Where Drupal says each branch and tag name should be prefixed with Drupal version. Eg: 8.x-1.x (for branch), and 8.x-1.0 (for tag release).

Please fix it first.
https://www.drupal.org/node/711070

sumit-k’s picture

Hi @rksyravi,

Thanks for review. Issue is fixed. Please review

sumit-k’s picture

Status: Needs work » Needs review
rksyravi’s picture

Issue summary: View changes
avpaderno’s picture

Priority: Critical » Normal
Issue summary: View changes
Issue tags: -PAreview: single application approval
avpaderno’s picture

Status: Needs review » Closed (duplicate)
Related issues: +#2976745: [D8] Multi step login

We just accept applications from users who don't have the vetted role. Any application created after the role is given is closed as Closed (duplicate) (if a previous application is found) or Closed (won't fix).

The purpose of these applications is not reviewing every project users create, but reviewing a single project created from users who aren't vetted users to make them able to opt into security review coverage for their (past and future) projects.

sumit-k’s picture

Alright. Got it. Thanks @kiamlaluno