Skip to content

feat(data-modeling): add note editing to collections and relationships COMPASS-9654 #7171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 7, 2025

Conversation

gribnoysup
Copy link
Collaborator

@gribnoysup gribnoysup commented Aug 6, 2025

This patch adds note editing to collections and relationships

image image

@github-actions github-actions bot added the feat label Aug 6, 2025
@gribnoysup gribnoysup added the feature flagged PRs labeled with this label will not be included in the release notes of the next release label Aug 6, 2025
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm moving all shared components here because otherwise we're still having slightly variable margings in sections, it's easier to control when this stuff is all coming from one place and the layout hierarchy is accounted for

@gribnoysup gribnoysup marked this pull request as ready for review August 7, 2025 08:52
@gribnoysup gribnoysup requested a review from a team as a code owner August 7, 2025 08:52
@@ -0,0 +1,26 @@
import { useState, useLayoutEffect } from 'react';

export function useChangeOnBlur(
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Anemy depending on which PR lands first, I think you might want to either copy this over or update your branch to use it

Copy link
Contributor

@paula-stacho paula-stacho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat! I'd just add some tests for the collection notes, especially cause it's a separate edit

@gribnoysup
Copy link
Collaborator Author

Good point, let me extend those!

@gribnoysup
Copy link
Collaborator Author

Units are passing and last commit didn't really changed anything else much, so I'll merge

@gribnoysup gribnoysup merged commit 88b2077 into main Aug 7, 2025
23 of 24 checks passed
@gribnoysup gribnoysup deleted the COMPASS-9654 branch August 7, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat feature flagged PRs labeled with this label will not be included in the release notes of the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants