Skip to content

Feature/post message to react native on thumbs up#24

Merged
jr-abtion merged 5 commits into
mainfrom
feature/post-message-to-react-native-on-thumbs-up
Apr 29, 2026
Merged

Feature/post message to react native on thumbs up#24
jr-abtion merged 5 commits into
mainfrom
feature/post-message-to-react-native-on-thumbs-up

Conversation

@jr-abtion

Copy link
Copy Markdown
Contributor

No description provided.

React native app renders this in a web view so it will inject the
window.ReactNativeWebView object. If this object is present we will use
it to send a message to the app.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a CLAUDE.md file to provide comprehensive guidance for Claude Code, detailing the project's architecture, commands, and environment configuration. In widget/index.html, the code underwent significant formatting cleanup and added a mechanism to notify React Native WebViews of conversation feedback. Feedback was provided to improve the robustness of the WebView message dispatching by verifying the existence of the postMessage function before invocation.

Comment thread widget/index.html
By mounting a volume the container will load the actual file being
modified by the IDE allowing it to hot reload changes without having to
run compose down and up to see every change
@jr-abtion jr-abtion merged commit 8f8b17d into main Apr 29, 2026
1 check passed
@jr-abtion

Copy link
Copy Markdown
Contributor Author

Client retracted on the use of this feature because they are not sure it will improve their app ratings to ask for a rating when the user is experiencing something confusing that they have to use the AI-Helper. However, I will not spend time removing this because it is non intrusive and small

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.

1 participant