Skip to content

Commit 943c31b

Browse files
committed
add ticket numbers for follow ups
1 parent 57a3ae3 commit 943c31b

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/compass-data-modeling/src/components/drawer/collection-drawer-content.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ const CollectionDrawerContent: React.FunctionComponent<
127127
const namespaces = useMemo(() => {
128128
return collections.map((c) => c.ns);
129129
}, [collections]);
130-
const database = useMemo(() => toNS(namespaces[0]).database, [namespaces]); // TODO: what if there are no namespaces?
130+
const database = useMemo(() => toNS(namespaces[0]).database, [namespaces]); // TODO(COMPASS-9718) use diagram.database
131131

132132
const { value: collectionName, ...nameInputProps } = useChangeOnBlur(
133133
toNS(namespace).collection,

packages/compass-e2e-tests/helpers/selectors.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1450,9 +1450,9 @@ export const DataModelZoomOutButton = `${DataModelEditor} [aria-label="Minus Ico
14501450
export const DataModelZoomInButton = `${DataModelEditor} [aria-label="Plus Icon"]`;
14511451
export const DataModelPreview = `${DataModelEditor} [data-testid="model-preview"]`;
14521452
export const DataModelPreviewCollection = (collectionId: string) =>
1453-
`${DataModelPreview} [data-id="${collectionId}"]`; // TODO: add once we upgrade reactflow again in diagramming: [aria-roleDescription="node"]
1453+
`${DataModelPreview} [data-id="${collectionId}"]`; // TODO(COMPASS-9719): add once we upgrade reactflow again in diagramming: [aria-roleDescription="node"]
14541454
export const DataModelPreviewRelationship = (relationshipId: string) =>
1455-
`${DataModelPreview} [data-id="${relationshipId}"]`; // TODO: add once we upgrade reactflow again in diagramming: [aria-roleDescription="edge"]
1455+
`${DataModelPreview} [data-id="${relationshipId}"]`; // TODO(COMPASS-9719): add once we upgrade reactflow again in diagramming: [aria-roleDescription="edge"]
14561456
export const DataModelApplyEditor = `${DataModelEditor} [data-testid="apply-editor"]`;
14571457
export const DataModelEditorApplyButton = `${DataModelApplyEditor} [data-testid="apply-button"]`;
14581458
export const DataModelUndoButton = 'button[aria-label="Undo"]';

0 commit comments

Comments
 (0)