Skip to content

Add relations in database event trigger output data #11820

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

martmull
Copy link
Contributor

Before

Enregistrement.de.l.ecran.2025-04-30.a.17.00.25.mov

After

Enregistrement.de.l.ecran.2025-04-30.a.16.58.38.mov

@martmull martmull marked this pull request as draft April 30, 2025 15:03
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Added support for relation fields in database event trigger output data, enabling workflows to access and manipulate related object data through nested structures.

  • Modified shouldGenerateFieldFakeValue.ts to include relation fields by removing the relation type exclusion check
  • Added depth-limited recursion in generate-fake-object-record.ts to prevent infinite loops when handling relation fields
  • Added fields.relationTargetObjectMetadata.fields to relations array in workflow-schema.workspace-service.ts for fetching complete object metadata
  • Potential inconsistency between computeDatabaseEventTriggerOutputSchema and computeRecordOutputSchema methods in handling relations needs review

4 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

Copy link
Contributor

github-actions bot commented Apr 30, 2025

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:33251

This environment will automatically shut down when the PR is closed or after 5 hours.

@martmull martmull force-pushed the 789-add-relations-field-type-variables-support-in-workflows branch from d9bac52 to 6125c7a Compare April 30, 2025 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

➕ Add Relations field type variables support in Workflows
2 participants