deep-foundation / deep-foundation/deeplinks

Auto-contain

Open
#212 1 comment 6 reactions 0 assignees View on GitHub
enhancement
Dominant language
TypeScript
Stars
13
Forks
8
PR merge metrics
No merged PRs in 30d

Description

### Is your feature request related to a problem? Please describe.

Add ability to pass containerLinkId to insert operations

### Describe the solution you'd like

Options:
1.
```
if (containerLinkId === null) do not contain
else if (containerLinkId === undefined) contain in `deep.linkId`
else contain in containerLinkId
```
2.
```
if (containerLinkId === undefined) do not contain
else contain in containerLinkId
```

I like the first solution because there are more situations when we contain links than situation when we do not contain links. It would be fine if be default a link will be contained in deep.linkId or in passed containedLinkId

### Describe alternatives you've considered

_No response_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.