Skip to content
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

Saved query logging #2137

Merged
merged 30 commits into from
Jan 6, 2022
Merged

Saved query logging #2137

merged 30 commits into from
Jan 6, 2022

Conversation

jrchudy
Copy link
Member

@jrchudy jrchudy commented Nov 3, 2021

This PR includes the logging improvements for the saved query functionality for the actions associated with saving queries and viewing the set to apply one on recordset. More details about the logging changes/additions can be found in the attached issue (#2122 )

Also included in this PR:

  • Tooltip for the saved query dropdown button that differs when disabled
  • default saved query names will no longer end in a ; character
  • Updated query parameters documentation to include other undocumented query parameters
  • updated logging.md to include new properties being logged and updated pactions
  • added pcid and ppid to recordedit link that redirects after a successful submission

…in github. Add 3 chaise config properties for the thresholds. Add a comment describing the behavior of default name and description.
…er to edit the existing query or cancel to modify the facet criteria
…ert to allow for tracking alerts in their own array separate from the main page
…ion to allow for an external close function to remove alerts from an external alerts array that isn't controled by the alerts service
@jrchudy jrchudy self-assigned this Nov 3, 2021
@jrchudy jrchudy requested a review from RFSH December 18, 2021 01:13
common/table.js Outdated Show resolved Hide resolved
docs/user-docs/logging.md Outdated Show resolved Hide resolved
@jrchudy jrchudy merged commit 276e6a8 into master Jan 6, 2022
@jrchudy jrchudy deleted the saved-query-logging branch February 16, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants