Check if every dataset in `web_tool_script_3.R` actually needs `investor_name` and `portfolio_name`

[jdhoffa]

• depends on Review all prep_* functions use of investor_name and portfolio_name, and if it's only needed in an initial filter, then remove  pacta.portfolio.report#34

And remove the call to add_inv_and_port_names_if_needed() if possible.

e.g.

workflow.transition.monitor/web_tool_script_3.R

Line 124 in 5430221

equity_results_map <- add_inv_and_port_names_if_needed(equity_results_map)

[cjyetman]

I think this would depend on https://github.com/RMI-PACTA/pacta.interactive.report/issues/199 being fixed/implemented, but good to have an issue to track this potential change in the future

[jdhoffa]

The repository that you link has been archived?

If that issue is important, maybe we should transfer it to wherever it should live?

[jdhoffa]

Ah never mind, here it is: RMI-PACTA/pacta.portfolio.report#34

(cant transfer private issues to a public repo)