Skip to content
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

refactor(form-builder): simplify cross dataset reference config (#3670) #3811

Merged
merged 5 commits into from
Nov 1, 2022

Conversation

sjelfull
Copy link
Member

@sjelfull sjelfull commented Oct 27, 2022

Porting #3670 to v3

  • refactor(form-builder): remove token and project id support in cross dataset references

  • refactor(base): remove token header from preview requests

  • refactor(schema): remove project and token validation from cross dataset reference schema

  • refactor(schema): derive search fields from preview definition for cross dataset references

  • refactor(desk-tool): remove cross dataset reference tokens from confirm delete dialog requests

  • chore(test-studio): update cross dataset reference schema in test studio

  • test(form-builder): remove project and experimental search from cross dataset reference test

  • refactor(types): remove token and project id support in cross dataset references

  • fix(desk-tool): replace projects terminology in confirm delete dialog

[sc-25932]

  • fix(form-builder): remove extra whitespace on reference inputs when empty

[sc-25936]

  • fix(form-builder): fix inconsistent padding on CDR preview

[sc-25927]

  • fix(desk-tool): handle unavailable dataset names in delete dialog

[sc-25932]

  • fix(form-builder): update inconsistent copy for missing document

[sc-25929]

  • fix(form-builder): handle invalid image sources in preview

  • fix(form-builder): limit CDR warning tooltip width

  • fix(form-builder): fix focus handling when editing array reference

  • feat(form-builder): show warning if cdr feature is disabled

  • fix(form-builder): removed cached inital state

  • refactor(form-builder): remove unnecessary conditional

What to review

Notes for release

@sjelfull sjelfull self-assigned this Oct 27, 2022
@vercel
Copy link

vercel bot commented Oct 27, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 1, 2022 at 1:34PM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
studio-workshop ⬜️ Ignored (Inspect) Nov 1, 2022 at 1:34PM (UTC)

@sjelfull sjelfull changed the base branch from chore/sc-21163/cherry-pick-changes-for-v3-on-next to v3 October 31, 2022 12:07
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch from 58a245b to d456e73 Compare October 31, 2022 13:42
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch 2 times, most recently from 795cfb2 to 8c7bbda Compare October 31, 2022 14:32
@sjelfull sjelfull marked this pull request as ready for review October 31, 2022 14:32
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch from 8c7bbda to b3cd5ec Compare October 31, 2022 14:46
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch from b3cd5ec to d6fa073 Compare October 31, 2022 15:07
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch from 99f8c5d to 65edd1d Compare November 1, 2022 10:13
Copy link
Member

@bjoerge bjoerge left a comment

Choose a reason for hiding this comment

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

Great work @sjelfull, LGTM!

sjelfull and others added 5 commits November 1, 2022 14:27
* refactor(form-builder): remove token and project id support in cross dataset references

* refactor(base): remove token header from preview requests

* refactor(schema): remove project and token validation from cross dataset reference schema

* refactor(schema): derive search fields from preview definition for cross dataset references

* refactor(desk-tool): remove cross dataset reference tokens from confirm delete dialog requests

* chore(test-studio): update cross dataset reference schema in test studio

* test(form-builder): remove project and experimental search from cross dataset reference test

* refactor(types): remove token and project id support in cross dataset references

* fix(desk-tool): replace projects terminology in confirm delete dialog

[sc-25932]

* fix(form-builder): remove extra whitespace on reference inputs when empty

[sc-25936]

* fix(form-builder): fix inconsistent padding on CDR preview

[sc-25927]

* fix(desk-tool): handle unavailable dataset names in delete dialog

[sc-25932]

* fix(form-builder): update inconsistent copy for missing document

[sc-25929]

* fix(form-builder): handle invalid image sources in preview

* fix(form-builder): limit CDR warning tooltip width

* fix(form-builder): fix focus handling when editing array reference

* feat(form-builder): show warning if cdr feature is disabled

* fix(form-builder): removed cached inital state

* refactor(form-builder): remove unnecessary conditional

Co-authored-by: Espen Hovlandsdal <espen@hovlandsdal.com>
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
* refactor(form-builder): remove token and project id support in cross dataset references

* refactor(base): remove token header from preview requests

* refactor(schema): remove project and token validation from cross dataset reference schema

* refactor(schema): derive search fields from preview definition for cross dataset references

* refactor(desk-tool): remove cross dataset reference tokens from confirm delete dialog requests

* chore(test-studio): update cross dataset reference schema in test studio

* test(form-builder): remove project and experimental search from cross dataset reference test

* refactor(types): remove token and project id support in cross dataset references

* fix(desk-tool): replace projects terminology in confirm delete dialog

[sc-25932]

* fix(form-builder): remove extra whitespace on reference inputs when empty

[sc-25936]

* fix(form-builder): fix inconsistent padding on CDR preview

[sc-25927]

* fix(desk-tool): handle unavailable dataset names in delete dialog

[sc-25932]

* fix(form-builder): update inconsistent copy for missing document

[sc-25929]

* fix(form-builder): handle invalid image sources in preview

* fix(form-builder): limit CDR warning tooltip width

* fix(form-builder): fix focus handling when editing array reference

* feat(form-builder): show warning if cdr feature is disabled

* fix(form-builder): removed cached inital state

* refactor(form-builder): remove unnecessary conditional

Co-authored-by: Espen Hovlandsdal <espen@hovlandsdal.com>
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
Signed-off-by: Fred Carlsen <fred@sjelfull.no>
@sjelfull sjelfull force-pushed the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch from 65edd1d to 7748411 Compare November 1, 2022 13:27
@sjelfull sjelfull merged commit 36419e2 into v3 Nov 1, 2022
@sjelfull sjelfull deleted the chore/sc-21163/cherry-pick-cdr-updates-for-v3-on-next branch November 1, 2022 13:27
@bjoerge bjoerge added the v3 label Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants