The "overall" results weblink for the EJAM Report column is currently a generic link, not a link to recreate the multisite report.
(Note a generic link is provided there because it is not likely a user with the app open or looking at the excel table would need to rerun the same analysis. maybe useful if a user saved or was sent the excel results but did not save or get sent the html/pdf report, but hard to build a link there that could recreate the full multisite analysis, so that was not done).
However, that generic link was set to the API homepage which is more technical than most users want. It is confusing for most users to click "EJAM Report" on the overall tab and get sent to a technical page about the API.
Instead,
-
When the analysis was for 2+ sites, ie a true multisite analysis the generic URL for the overall results "EJAM Report" column should be set to the EJAM app overall, like url_package("ejamapp") (not the API homepage)
-
When the analysis was of just one site, however, that is a special case since the overall and bysite results are essentially the same and the link to "EJAM Report" should be just set to be the same in the overall table as it is in the bysite table.
This should be changed maybe in ejamit() where it does links <- url_columns_bysite( or maybe within url_columns_bysite() which creates the multiple columns of various kinds of reports links, or maybe in the more basic helper that creates just the EJAM Report links which seems to be url_ejamapi() . It would be easiest to change the default in url_ejamapi(). But the logic for knowing if URLs are being provided for the overall results table and knowing if it was a 1-site analysis would have to be in either ejamit() or maybe url_columns_bysite() , I think.
Note this should improve both the webpage table of detailed results by site and the excel downloaded version of that table, not just the output of ejamit().
The "overall" results weblink for the EJAM Report column is currently a generic link, not a link to recreate the multisite report.
(Note a generic link is provided there because it is not likely a user with the app open or looking at the excel table would need to rerun the same analysis. maybe useful if a user saved or was sent the excel results but did not save or get sent the html/pdf report, but hard to build a link there that could recreate the full multisite analysis, so that was not done).
However, that generic link was set to the API homepage which is more technical than most users want. It is confusing for most users to click "EJAM Report" on the overall tab and get sent to a technical page about the API.
Instead,
When the analysis was for 2+ sites, ie a true multisite analysis the generic URL for the overall results "EJAM Report" column should be set to the EJAM app overall, like url_package("ejamapp") (not the API homepage)
When the analysis was of just one site, however, that is a special case since the overall and bysite results are essentially the same and the link to "EJAM Report" should be just set to be the same in the overall table as it is in the bysite table.
This should be changed maybe in
ejamit()where it doeslinks <- url_columns_bysite(or maybe withinurl_columns_bysite()which creates the multiple columns of various kinds of reports links, or maybe in the more basic helper that creates just the EJAM Report links which seems to be url_ejamapi() . It would be easiest to change the default in url_ejamapi(). But the logic for knowing if URLs are being provided for the overall results table and knowing if it was a 1-site analysis would have to be in either ejamit() or maybe url_columns_bysite() , I think.Note this should improve both the webpage table of detailed results by site and the excel downloaded version of that table, not just the output of ejamit().