CUNY Academic Commons

Thanks Laurie, I am adding Sara as a watcher and adding this to our agenda for Friday.

About the "Manage > Digital Research Tools" menu, we still have the Tapor Client plugin activated, which is why this menu item is showing up. Boone, do we still need this plugin activated? If so, I can write a code snippet to remove this menu item. If not, let's deactivate the Tapor Client plugin.

For the new styling for the Groups submenu, we would probably want to mirror the same change to the Users submenu too right?

Looks great to me!

I agree -- these look great!

I've committed the new submenu design to a new Git branch: feature/22069-new-submenu. Once we have a dev site up-and-running, we can get the team to test out the change.

In the meantime, I've removed the "Manage > Digital Research Tools" tab from the group tabs in https://github.com/cuny-academic-commons/cac/commit/415b2fbf654fd719e96985e92b8a9830634901cc.

Also adding Ray and Boone as watchers.

I had to switch over the dev site to the 2.5.x branch to do some work with Reclaim regarding replication, so what you're seeing probably doesn't include Ray's changes. I can't make the switchover right at the moment - Ray, feel free to switch the branch if you'd like, otherwise I'll do it in about an hour when I can hop on the VPN.

Is this updated submenu design available to test on the dev site now? Bumping in particular because I noticed some strange formatting on the Manage tab just now (see attached) in an incognito window in Chrome.

Are you seeing this display bug on production or on the dev site, Colin? I'm not seeing this on production.

On cdev, I've just merged the new submenu design into 2.6.x branch and is now available for testing. You might need to purge your browser cache to see the new changes. Let me know if anyone finds any display issues with the new design.

Thanks for testing, Colin.

I was able to duplicate the problem when I logged in as a regular member and not as a super admin. The problem is tied to #23101. In the meantime, I've added the proposed workaround from #23101 on cdev so you can test the submenu design without the display bug.

Thanks Ray, I just took another look and the workaround and submenu redesign both seem to be working well. I'll mention this in the community call on Friday for further testing, but especially given the issue on production, it seems to me we could push all of this sooner rather than later, whenever you think works best.

Any thoughts on why I'm getting a 403 error trying to go incognito on the dev site?

I've just pushed a fix, which is part of a fix to address #22668 and #23101, that should also address the group "Manage" submenu display bug on production.

We can push the tabbed submenu change in the next maintenance release, if others are okay with that.

About:

Any thoughts on why I'm getting a 403 error trying to go incognito on the dev site?

Can you list some steps to duplicate the issue? On cdev, which URLs are you experiencing 403s with?

Thanks for that immediate fix Ray, and sounds good about the tabbed change. I'll get some more testing done on it. Sara, maybe you can take a look on dev?

You can disregard my thing about 403s in incognito. I think I was doing something silly before.

Thanks Ray, that sounds good. I spoke to Scott on our Friday call last week about testing this and reporting back to this ticket when he can. Adding him as a watcher here.