I'm having some difficulty reading the names that have both id and locator in them. I think it would be better to just drop the locator part of the name, and to just document that locators are used as the id of static pages. IMO it would be easier to read as requested_course_tab_ids.