-
Notifications
You must be signed in to change notification settings - Fork 20
Replace admin-only broken link filter with user-accessible sidebar toggle #1177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: fbacall <503373+fbacall@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot can you add documentation for this filter in the OpenAPI specs: public/api/definitions/tess.yml ?
fbacall
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot can you add documentation for this filter in the OpenAPI specs: public/api/definitions/tess.yml ?
…terials Co-authored-by: fbacall <503373+fbacall@users.noreply.github.com>
Summary of changes
include_broken_links)searchable.rbinclude_expiredandinclude_disabledinclude_broken_linksquery parameter in the OpenAPI specs (public/api/definitions/tess.yml)Motivation and context
Fixes #1176
Previously, resources with broken links were automatically hidden via a hardcoded Solr facet in
searchable.rb(lines 108-113) unless the user was an admin. This made broken links invisible and undiscoverable to regular users.The new implementation:
include_broken_linksas a special facet inlib/facets.rbapplicable to models withlink_monitorfailingboolean field in Solr/eventsand/materialsendpointsScreenshots
(N/A - UI change follows existing toggle pattern)
Checklist
to license it to the TeSS codebase under the
BSD license.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.