Conversation
Exclude from: - API queries - Tag searches - Homepage featured profiles - School profile list - Browse all profiles list (when not searching) Unlisted profiles can still show up when using the main site search box.
Closed
Contributor
There was a problem hiding this comment.
- Verified that unlisted profiles records are not included in the 'Browse All' results (empty search parameters), but they can still appear when searching by name.
- Verified that unlisted profiles records are not included in the search results with a matching tag, but they can appear if they contain a matching keyword.
Contributor
There was a problem hiding this comment.
- Verified that API index endpoint does not included unlisted profiles records.
- Verified that unlisted profiles records are still accessible individually via API.
Contributor
There was a problem hiding this comment.
- Verified that unlisted profiles records are not included in the profiles results filtered by school.
Contributor
There was a problem hiding this comment.
- Type filter works as expected.
Contributor
There was a problem hiding this comment.
- Verified that profiles names of the type 'unlisted' or 'in memoriam' are not included in the profile picker field in the student research app form.
Contributor
There was a problem hiding this comment.
- The
updateAdvancedmethod works as expected, allowing only users with the profile editor role from the same school or department to edit the profile type.
| @@ -18,24 +18,33 @@ | |||
| <label for="publicFilter">Public</label> | |||
Contributor
There was a problem hiding this comment.
- Suggestion: For consistency (even though this is a page for admin access only), this label should read 'Visibility' instead of 'Public' as the table header.
Contributor
There was a problem hiding this comment.
With an authorized role for this action, the edit view allows editing this setting. After removing the role in another tab and attempting to save the changes in the first tab, it returns a 403 error, preventing the action from completing.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add Profile types: