-
Notifications
You must be signed in to change notification settings - Fork 7
Issue 51727: Metrics to count the usage of webparts #6091
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
Closed
labkey-jeckels
wants to merge
36
commits into
release24.11-SNAPSHOT
from
24.11_fb_51727_webpartMetrics
Closed
Issue 51727: Metrics to count the usage of webparts #6091
labkey-jeckels
wants to merge
36
commits into
release24.11-SNAPSHOT
from
24.11_fb_51727_webpartMetrics
Conversation
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
* Issue 50861: Pass around properly encoded WebDav URLs * Fix rename to names with special characters Misc NAb-related fixes * Test fixes * Misc cleanup * Fix FilesWebpartFileRootTest * More consistent encoding, fix typo * Remove debugger statement * Revert to storing relative paths non-URI-encoded for file web parts * Fix encoding of paths on client * Simplify URI creation * PageFlowUtil: better match browser encodeURI, encodeURIComponent * Follow-up changes for encoding * Retain PageFlowUtil.encode() implementation - separate from expectation that it is "like" JavaScript encodeURIComponent() - retain encoding single-quote in encodeURIComponent() * File Browser: display decoded path for "WebDav URL" * Touch (to initiate CI build) --------- Co-authored-by: labkey-tchad <[email protected]> Co-authored-by: labkey-nicka <[email protected]>
# Conflicts: # core/package-lock.json # core/package.json # experiment/package-lock.json # experiment/package.json
…e, label, or import alias (#6025) - currently only supports name
- ParameterCurveFit to support constants for min and max asymptote - Add Sigmoidal Logistic 3 Parameter curve fit - StatsServiceImpl.TestCase update for 3PL curve fit and 4PL simplex fit - Add curve fit stats for new API: residual sum of squares, root mean square error, R2, adjusted R2
…ociate fcs files with the wsp (#6074) Add check to prevent possible NPE
labkey-adam
approved these changes
Nov 22, 2024
| new SqlSelector(getSchema(), | ||
| sql).forEach(rs -> { | ||
| counts.put(rs.getString("Name"), rs.getInt("C")); | ||
| } |
Contributor
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.
Could use getValueMap(), but whatever
* Update to Manage "Participant" link and text on Manage Study page. * Add 'Delete Participant' section. * Implement 'Delete' Action. * Handle typeahead, dynamic combobox load, fix delete from ParticipantGroupMap. * Add 'View Participant Data' link. * Error handling and log messages. * Add audit log for deleting from ParticipantGroupMap table. --------- Co-authored-by: Lum <[email protected]>
# Conflicts: # api/src/org/labkey/api/ApiModule.java # experiment/package-lock.json # experiment/package.json
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Rationale
We'd like insights into which web parts are being used and which can be deleted.
Changes
webPartCountsmetric, with each web part name as a child and its usage count as its value