Skip to content

Conversation

@chrismckiernan
Copy link
Contributor

No description provided.

@chrismckiernan chrismckiernan changed the title Feature 2363/email guild leaders Email added/updated guild leaders about their assignment Jun 3, 2024
Copy link
Contributor

@vhscom vhscom left a comment

Choose a reason for hiding this comment

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

Looks like we're calling an undefined variable when attempting to log which would throw an error in the browser. Our linter doesn't pick up on this currently and because we're not strongly typed it's easy to miss as the type of log is any per static analysis.

…omputing/check-ins into feature/2363-email-guild-leaders
@chrismckiernan chrismckiernan force-pushed the feature-2363/email-guild-leaders branch from 96a81d6 to 128e161 Compare June 3, 2024 21:24
@chrismckiernan chrismckiernan requested a review from vhscom June 3, 2024 21:25
Copy link
Contributor

@vhscom vhscom left a comment

Choose a reason for hiding this comment

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

:shipit:

@chrismckiernan chrismckiernan merged commit c7aa4d7 into develop Jun 3, 2024
@chrismckiernan chrismckiernan deleted the feature-2363/email-guild-leaders branch June 3, 2024 21:56
@timyates
Copy link
Contributor

timyates commented Jun 4, 2024

🎉 Should we close #2363 as completed by this?

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.

5 participants