Skip to content

Feedback on adding TES links to lots-of-links backend

Feedback in the form of code.

The first two links could be directly included in the vector, made available to everyone viewing the TES JSON, because access to the TES widget itself already filters down to people for whom TES is relevant. So then the backend just needs to do extra filtering for the Madison-specific link.

If we were going to do the filtering for the first two links, we'd have to filter on something other than ismemberof since non-Madison users don't in general get ismemberof headers. (So, we'd filter on eppn suffix).

But simpler to just straight up not have to do that filtering, figuring the audience filtering on access to the widget at all took care of that.

Merge request reports