Skip to content

Conversation

@tdari
Copy link
Contributor

@tdari tdari commented Jan 11, 2026

Hey,

I simply added an extra join to get the username field from the users table and now we return that username instead of the ingestion_schedules.created_by value. Resolves #54

image image

@tdari tdari marked this pull request as draft January 11, 2026 17:34
@tdari tdari force-pushed the created-by-username branch from e054ddc to 758d165 Compare January 11, 2026 17:51
@tdari tdari marked this pull request as ready for review January 11, 2026 17:56
Copy link
Member

@charlie-haley charlie-haley left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing this @tdari 🎉

@charlie-haley charlie-haley merged commit 2d91741 into marmotdata:main Jan 11, 2026
5 of 6 checks passed
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.

using the user name field instead of the user id field in runs history

2 participants