WIP DetailPage uber component

[ir4y]

No description provided.

[vesnushka]

To proper display tabs on the page you need to place it to headerContent of PageContainer

<PageContainer
  title={getTitle(resource!)}
  layoutVariant="with-tabs"
  headerContent={
    <PageTabs tabs={tabs} basePath={basePath} />
  }
>

[ruscoder]

Params should be more specific because you're hard-coded to id below

const menuItems: RouteItem[] = tabs.map(({ label, path }) => ({
        label,
        path: `/${basePath}/${params.id}/${path}`,
});

I suggest using at least. Params & {id: string}

I think something like

[ruscoder]

Do we need basePath here? Maybe we can use relative paths in links? e.g. ../path

[ruscoder]

I'm thinking about having uniform interface like in the ResourceListPage, when we pass {resource:R , bundle: Bundle} to the component. What do you think?

[ruscoder]

Also getTitle might have the similar interface ({resource,bundle}) for some use-cases (e.g. when we load Patient and we need to display it's managing organization)

[ruscoder]

I don't think we should store currentPath in the state if we have location and we subscribed to it using useLocation. Just use location.pathname instead of currentPath

[pavlushkin]

• Use custom renderLoading on uber details page #438

[pavlushkin]

I suggest to add a new prop getHeaderActions for DetailPage.
A real feature I'd like to implement is to add a navigation action like: