Skip to content

Commit 1bd36d8

Browse files
committed
Replace report url placeholders with proper filters
1 parent 08572ce commit 1bd36d8

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/browsergym/workarena/tasks/dashboard.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
REPORT_RETRIEVAL_MINMAX_CONFIG_PATH,
2121
REPORT_RETRIEVAL_VALUE_CONFIG_PATH,
2222
REPORT_DATE_FILTER,
23+
REPORT_TIME_FILTER,
2324
REPORT_PATCH_FLAG,
2425
)
2526
from ..instance import SNowInstance
@@ -305,7 +306,9 @@ def setup_goal(self, page: playwright.sync_api.Page) -> Tuple[str | dict]:
305306
)
306307
# ... set start URL based on config
307308
# ...... some of the reports have need a date filter to be applied so we do this by patching a placeholder in the URL
308-
self.start_url = self.instance.snow_url + self.config["url"]
309+
self.start_url = self.instance.snow_url + self.config["url"].replace(
310+
"REPORT_DATE_FILTER", REPORT_DATE_FILTER
311+
).replace("REPORT_TIME_FILTER", REPORT_TIME_FILTER)
309312

310313
# Produce goal string based on question type
311314
chart_locator = (
@@ -678,7 +681,7 @@ def _generate_random_config(
678681
# On the fly generated report
679682
if not report.get("sys_id", None):
680683
# ... these receive a filter that is added through the URL
681-
url = f"/now/nav/ui/classic/params/target/sys_report_template.do%3Fsysparm_field%3D{report['field']}%26sysparm_type%3D{report['type']}%26sysparm_table%3D{report['table']}%26sysparm_from_list%3Dtrue%26sysparm_chart_size%3Dlarge%26sysparm_manual_labor%3Dtrue%26sysparm_query=sys_created_on<javascript:gs.dateGenerate('{REPORT_DATE_FILTER}','00:00:00')^EQ"
684+
url = f"/now/nav/ui/classic/params/target/sys_report_template.do%3Fsysparm_field%3D{report['field']}%26sysparm_type%3D{report['type']}%26sysparm_table%3D{report['table']}%26sysparm_from_list%3Dtrue%26sysparm_chart_size%3Dlarge%26sysparm_manual_labor%3Dtrue%26sysparm_query=sys_created_on<javascript:gs.dateGenerate('{REPORT_DATE_FILTER}','{REPORT_TIME_FILTER}')^EQ"
682685
# Report from the database
683686
else:
684687
url = f"/now/nav/ui/classic/params/target/sys_report_template.do%3Fjvar_report_id={report['sys_id']}"

0 commit comments

Comments
 (0)