CDLUC3 / dmptool_late_2019

Home of the DMP Roadmap-based version of the DMPTool
MIT License
24 stars 6 forks source link

DMPTool image for IdPs missing #249

Closed cpwillett closed 5 years ago

cpwillett commented 5 years ago

Some IdPs display the DMPTool logo; it's a broken link now. It looks like the link is URL encoded in the HTML: https://dmptool.org/images/DMPTool_logo_blue_shades_v1b3b.svg

See attached png for a screen shot. This is a repeat of issue 126 from a few years ago--seems to have reappeared with the latest upgrade.

dmptool-idp

marisastrong commented 5 years ago

what are the steps needed to recreate the problem? Is this when a user logs in? or when using the printenv tool?

briri commented 5 years ago

I manually copied the logo to the appropriate directory and updated the deploy script to do this going forward.

Please have them test it again @cpwillett

cpwillett commented 5 years ago

Seems to work now. Anyone can see it, not just people at these institutions. When you go to sign in, click on Your Institution, type in "Texas", choose University of North Texas and click Go. You'll go to the UNT IdP and can see the logo (now). Not all IdPs include the logo, so you have to hunt around. Looking back at the previous ticket, it looks like UNT, University of Texas at Dallas, and Woods Hole (WHOI) are three that do, so they're good to test with. Thanks for fixing it!