Content Usage Metrics Dashboard #79

toph-allen · 2025-04-25T00:07:29Z

The long foretold "megadashboard".

This dashboard merges the "outer" most-used-content and inner who-is-viewing-this-content dashboards. It contains all the features and functionality from those dashboards.

You can find links to view the dashboard in the attached issue.

The code is pretty complex, but I went through and did a close read at the end of the process to try to organize it in a way that makes sense, erring on the side of verbose comments.

Features:

View metrics.
Click on a content item to navigate to a detail view, click the Back button to return to the overview.
Select a date range to view, adjust visit merge window. These controls operate both on the outer list and the inner detail view.
View-specific controls:
- The outer list lets you filter by content type, show/hide the GUID column, and export data.
- The inner view lets you filter by user and send an email to all selected users.
Shiny Bookmarking is enabled: navigating directly to a content item's URL (when in "open solo" mode) will load that content item's data*
Improved user experience upon initial deploy / when no Viewer API Key integration is selected: Instead of crashing, the app displays an undismissable modal dialog with setup instructions.

* I couldn't get this to actually update the browser history entry, so your browser's back button won't work to navigate back.

Bugs / new features to fix:

When navigating "back", state in conditional sidebar filters is lost.
Bookmarking is not working (restore does not work)
Fail gracefully on first deploy (display a specific screen when client creation fails due to missing integration)

Subsequent issue:

Content scope control: https://github.com/posit-dev/connect/issues/31167

- bookmarking - state preservation of conditional controls - better first run (no integration) experience

dotNomad

I'm not sure if this would be helpful, but now that these are combined I miss the supplying the GUID/URL functionality. I wonder if we can bring that back in someway?

We can search the content (can't search the GUID though). You may have an idea of how that could be incorporated again.

Fantastic work. I had a couple things (mainly CSS) that I had questions on. I did find I missed the URL/GUID input method if I knew what I wanted to look at immediately. Getting this into a megadashboard is fantastic.

extensions/who-is-visiting-this-content/www/styles.css

dotNomad · 2025-04-25T23:28:24Z

extensions/who-is-visiting-this-content/www/styles.css

+#visit_merge_window_text {
+  margin-top: -10px !important;
+}


I never like seeing negative margin or !important. They have there uses, but this is usually a code smell for me.

Looks like this is getting around the bslib-gap-spacing but could we combine the Visit Merge Window bits into a single form group to avoid negative margin?

Yeah, this is trying to shrink the spacing between the text and slider input for the visit merge window.

Experimentally, the !important isn't required, but negative margin is probably still not ideal.

I think it would be much more fiddly to try to combine them (and I'm not even sure how that'd actually work in the R code). The default styling applied to sidebar input items is only applied to the direct children of the sidebar, and I've had a bunch of other back and forth with this merge working on getting elements to look and behave correctly (e.g. the auto-hiding elements — they were originally in a div and everything looked wrong).

I would also definitely be open to deleting the text input for the visit merge window. The text input was a request from Katie back when the maximum range was 600, and it was impossible to get consistent increments. The maximum range is now lower, and it's much easier to step the slider up and down by increments of 1. And the code to sync the slider and text input can still cause some finicky behavior in my experience.

dotNomad · 2025-04-25T23:32:23Z

extensions/who-is-visiting-this-content/app.R

+        HTML(paste(
+          "In the Access panel to the right, click <strong>\"Add integration\"</strong>,",
+          "then select a <strong>Visitor API Key</strong> integration.",
+          "If you don't see one in the list, an administrator must enable this feature on your Connect server.",
+          "See the <a href='https://docs.posit.co/connect/admin/integrations/oauth-integrations/connect/' target='_blank'>Admin Guide</a> for setup instructions.",
+          "<br><br>",
+          "For guidance on using visitor-scoped permissions in your own Connect apps, see the",
+          "<a href='https://docs.posit.co/connect/user/oauth-integrations/#obtaining-a-visitor-api-key' target='_blank'>User Guide</a>.",
+          sep = " "


This improvement is HUGE for Gallery users. 🎉

dotNomad · 2025-04-25T23:33:50Z

extensions/who-is-visiting-this-content/app.R

+
+  # Global UI elements ----
+
+  output$page_title_bar <- renderUI({


Perhaps we can provide a URL / GUID somewhere in this title bar?

I like this idea! I'll put the GUID there for now, as it seems like reasonable info to have while not being so long that it makes things difficult to lay out. (Plus, the Open in Connect button has the URL).

toph-allen · 2025-04-28T21:49:15Z

I'm not sure if this would be helpful, but now that these are combined I miss the supplying the GUID/URL functionality. I wonder if we can bring that back in someway?

We can search the content (can't search the GUID though). You may have an idea of how that could be incorporated again.

Fantastic work. I had a couple things (mainly CSS) that I had questions on. I did find I missed the URL/GUID input method if I knew what I wanted to look at immediately. Getting this into a megadashboard is fantastic.

@dotNomad I do kind of miss that functionality too. When I was initially thinking of adding it, I thought forward to how to add it in the combined dashboard. Talking to @jonkeane, we thought about using reactable's built in filters. (You can add custom filters that use arbitrary functions to filter the table, so we could process the entered text to extract any GUIDs and search multiple columns, I think.)

Some caveats:

Given that the GUID is hidden by default, would we use the Content column? How would you make it clear you could paste in a URL to filter the list?)
I was originally gonna write that it's too slow, but it seems faster now. Oh — maybe it's slow in Safari.

Two ideas:

We could have an external search box like in the Basic custom search method example, with placeholder text that says something like "Search title, GUID, or URL". I'm not sure if these can search hidden columns — but it looks like you might be able to.
We could have a button in the title bar on the outer screen that launches a modal to paste in a URL / GUID, and brings up the inner dashboard when clicked.

Initially I liked the idea of a button in the title bar, but filtering the table might be better.

Open questions:

Worth including vanity URLs? Might be a "nice to have" but ultimately much more fiddly to implement feature

toph-allen · 2025-04-28T23:20:27Z

@dotNomad I created a global search field that searches the title, GUID, dashboard URL, and owner username. (As a result, I removed the filter fields at the top of title and owner username, but could add them back.)

I can't figure out the CSS to make reactable's default search widget wider — any ideas @dotNomad?

Vanity URLs are not included currently, because of a connectapi issue: posit-dev/connectapi#398. I could add them in a more hacky way, but better to wait and do it the right way.

toph-allen added 5 commits April 24, 2025 20:02

no conditional sidebar behavior

f582d91

Use conditional UI for mutable controls

612d699

harmonize the inner and outer dashboard a bunch

bfad768

renv snapshot

bf6382a

remove reactlog from app

4964d2f

toph-allen mentioned this pull request Apr 25, 2025

content usage metrics dashboard #78

Closed

remove unnecessary tag

459be88

toph-allen requested review from jonkeane, fizzboop and dotNomad April 25, 2025 00:22

toph-allen added 2 commits April 25, 2025 12:45

many improvements

a33d119

- bookmarking - state preservation of conditional controls - better first run (no integration) experience

More quality of life & clarity updates

086da9a

toph-allen marked this pull request as ready for review April 25, 2025 18:52

toph-allen added 4 commits April 25, 2025 15:02

small language change

a6e0753

download button style tweaks

5155cc8

clean up imports

55fdf21

fix tidyr call, move full_url to function

318c936

toph-allen force-pushed the toph/content-usage-metrics-megadashboard branch from c3b8b88 to 318c936 Compare April 25, 2025 19:15

toph-allen added 2 commits April 25, 2025 16:31

re-organize, clarify, remove redundancy

b7f1e2f

trailing newline

55f48cc

toph-allen mentioned this pull request Apr 25, 2025

Add "Content Scope" control to the usage metrics dashboard #82

Open

toph-allen requested a review from cgraham-rs April 25, 2025 22:39

dotNomad previously approved these changes Apr 25, 2025

View reviewed changes

toph-allen dismissed dotNomad’s stale review via c87bbf5 April 28, 2025 23:08

respond to feedback

24b64d3

toph-allen force-pushed the toph/content-usage-metrics-megadashboard branch from c87bbf5 to 24b64d3 Compare April 28, 2025 23:18

toph-allen requested a review from dotNomad April 28, 2025 23:20

Update CSS for search bar

70d7903

toph-allen added 2 commits April 29, 2025 14:26

remove unnecessary css

4785c47

allow dashboard url superstrings to match

930143a

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Content Usage Metrics Dashboard #79

Content Usage Metrics Dashboard #79

toph-allen commented Apr 25, 2025 •

edited

Loading

dotNomad left a comment

dotNomad Apr 25, 2025

toph-allen Apr 28, 2025

dotNomad Apr 25, 2025

dotNomad Apr 25, 2025

toph-allen Apr 28, 2025

toph-allen Apr 28, 2025

toph-allen commented Apr 28, 2025 •

edited

Loading

toph-allen commented Apr 28, 2025 •

edited

Loading


		# Global UI elements ----

		output$page_title_bar <- renderUI({

Content Usage Metrics Dashboard #79

Are you sure you want to change the base?

Content Usage Metrics Dashboard #79

Conversation

toph-allen commented Apr 25, 2025 • edited Loading

dotNomad left a comment

Choose a reason for hiding this comment

dotNomad Apr 25, 2025

Choose a reason for hiding this comment

toph-allen Apr 28, 2025

Choose a reason for hiding this comment

dotNomad Apr 25, 2025

Choose a reason for hiding this comment

dotNomad Apr 25, 2025

Choose a reason for hiding this comment

toph-allen Apr 28, 2025

Choose a reason for hiding this comment

toph-allen Apr 28, 2025

Choose a reason for hiding this comment

toph-allen commented Apr 28, 2025 • edited Loading

toph-allen commented Apr 28, 2025 • edited Loading

toph-allen commented Apr 25, 2025 •

edited

Loading

toph-allen commented Apr 28, 2025 •

edited

Loading

toph-allen commented Apr 28, 2025 •

edited

Loading