Fix: showing current settings tab broken in MULTI_ORG.

[arikfr]

What type of PR is this? (check all applicable)

• Bug Fix

Description

Current logic of active tab selection assumed the path will begin with the path of the tab, but in MULTI_ORG the path begins with the org slug.

[kravets-levko]

There is simpler solution.

Add this import to client/app/components/SettingsWrapper.jsx:

import { stripBase } from "./Router";

And use it to prepare argument for settingsMenu.getActiveItem

redash/client/app/components/SettingsWrapper.jsx

Lines 12 to 14 in ae9bbe2

	return function SettingsTab(props) {
	const activeItem = settingsMenu.getActiveItem(location.path);
	return (

Probably a proper solution will be to change location service to strip base by default, but this should be fine for now.

[arikfr]

@kravets-levko thanks! Updated the implementation and added a test case.

[kravets-levko]

🚀