Skip to content

Conversation

alserene
Copy link
Contributor

@alserene alserene commented Oct 2, 2025

The fix was a missing _ in the field name.

@alserene alserene requested review from emanehab99 and Vebop October 2, 2025 04:35
Copy link
Contributor

@emanehab99 emanehab99 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 Alice!

Copy link
Contributor

@Vebop Vebop left a comment

Choose a reason for hiding this comment

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

good call moving the calculation up out of the return 👍

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.

4 participants