Following on from #3107196: Add 2-step login process, I've added a couple of incidental pieces to make the 2 available types of login a bit more accessible:

* custom block that exposes "Login with UserID" and "Login with Certificate" buttons.
* replacement Login/Logout menu link to replace core one with Certificate Login versions.

CommentFileSizeAuthor
#2 certificatelogin-links-3107197-1.patch4.88 KBspiderman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

spiderman created an issue. See original summary.

spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
CreditAttribution: spiderman at Consensus Enterprises commented
FileSize
certificatelogin-links-3107197-1.patch4.88 KB

Patch attached with both custom block and new menu item.

  • spiderman committed 1d7bcb9 on 8.x-1.x 
    Issue #3107197 by spiderman: Add custom block for certificatelogin links
  • spiderman committed e845c76 on 8.x-1.x
    Issue #3107197 by spiderman: Add replacement menu item for...
spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
CreditAttribution: spiderman at Consensus Enterprises commented

I've committed these as 2 distinct commits on the 8.x-1.x branch in prep for cutting a new alpha release.

bkraegelin’s picture

bkraegelin CreditAttribution: bkraegelin commented

I incorporated this patch in my website. But I get errors:

Uncaught PHP Exception Symfony\\Component\\Routing\\Exception\\RouteNotFoundException: "Route "certificatelogin.initial" does not exist." at /var/www/drupal8/web/core/lib/Drupal/Core/Routing/RouteProvider.php line 208

Any suggestions?

Thanks, Birger

spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
CreditAttribution: spiderman at Consensus Enterprises commented

@bkraegelin Apologies for the slow reply here- I haven't been actively working on this module lately. If you tried to apply this patch to the latest release, it's very likely that just won't work, as there's been a bunch of refactoring and other work the patch is based on. If at all possible, I'd suggest trying to swap in the 8.x-1.x-dev release (even if only in a dev environment) to see if it works any better.

If so, please report back! I'll happily cut a new release when I have some feedback to know it's working for others :)

Ambient.Impact’s picture

Ambient.Impact
they/them
Primary language English
Location Toronto
CreditAttribution: Ambient.Impact as a volunteer and at Consensus Enterprises commented

Can this issue be closed as it seems it's all committed?

bkraegelin’s picture

bkraegelin CreditAttribution: bkraegelin commented

Closed.
Issue resolved 2 years ago, but I don't remember how...

spiderman’s picture

spiderman
he/they
Primary language English
Location Halifax, NS
CreditAttribution: spiderman at Consensus Enterprises commented
Status: Needs review » Fixed

Thanks for the update @bkraegelin. Closing :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.