Skip to content

Conversation

@jrafanie
Copy link
Member

@jrafanie jrafanie commented Jan 7, 2026

  • To verify the response is valid on master, I replaced the 2016 version of the
    response with the current. It has the correct notifications description
    without the invalid escape sequence.

  • Fix tests to support new features from base entrypoint

To verify the response is valid on master, I replaced the 2016 version of the
response with the current.  It has the correct notifications description
without the invalid escape sequence.
@jrafanie jrafanie force-pushed the update-base-entrypoint-response branch from a4f004b to 16bdff8 Compare January 7, 2026 16:31
@jrafanie jrafanie changed the title [WIP] Update base entrypoint response Update base entrypoint response Jan 7, 2026
@jrafanie jrafanie removed the wip label Jan 7, 2026
Copy link
Member

@Fryguy Fryguy left a comment

Choose a reason for hiding this comment

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

I wonder why this isn't built dynamically or use method missing like the other classes

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.

2 participants