Adding a simpler "test" widget and server. #38
+413
−0
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
src/pyproject.toml
Outdated
There was a problem hiding this comment.
to make it consistent with other examples, could you remove the pyproject.toml + uv.lock & .python-version?
There was a problem hiding this comment.
Ok... I've removed these files.
test_server_python/README.md
Outdated
| @@ -0,0 +1,50 @@ | |||
| ## Known issues: | |||
There was a problem hiding this comment.
Can we remove this README?
There was a problem hiding this comment.
Hi Katia, @katia-openai
Given the latest merges to the openai repo - it looks like this info is no longer need.
I've removed the entire file.
test_server_python/README.md
Outdated
|
|
||
| ## Static files are used from the /static path on the MCP server | ||
|
|
||
| After running ```pnpm run build``` you need to copy all the test*.* files in the /assets directory to the /static folder under this server. |
There was a problem hiding this comment.
You shouldn't need to do that specifically for this example, just adding the app to build-all is enough.
There was a problem hiding this comment.
I happen to be copying the files over a static folder in the MCP server so that it can be served from the server instead using a cdn.
There was a problem hiding this comment.
Hi Katia, @katia-openai
Given the latest merges to the openai repo - it looks like this info is no longer need.
I've removed the entire file.
Remove 'Known issues'. I've fixed the issues.
|
Hi Katia,
I've updated the points mentioned.
I'm using the /static path on the MCP server in-place of using a cdn.
Greg
…On Wed, Oct 15, 2025 at 5:30 AM Katia Gil Guzman ***@***.***> wrote:
***@***.**** commented on this pull request.
Thanks for the PR! Added a few comments to make this example consistent
with the others
------------------------------
On src/pyproject.toml
<https://urldefense.com/v3/__https://github.com/openai/openai-apps-sdk-examples/pull/38*discussion_r2432364167__;Iw!!G92We9drHetJ8EofZw!dXSFSek9zUhCsT9ee7u3ygCtyfm2VRN8oCMdCbaR6TVdJn9RK8fAiAFDeGGyQx68RbYoIO_s5j6A4QLwJSXDE8OqNUg$>
:
to make it consistent with other examples, could you remove the
pyproject.toml + uv.lock & .python-version?
------------------------------
In test_server_python/README.md
<https://urldefense.com/v3/__https://github.com/openai/openai-apps-sdk-examples/pull/38*discussion_r2432367212__;Iw!!G92We9drHetJ8EofZw!dXSFSek9zUhCsT9ee7u3ygCtyfm2VRN8oCMdCbaR6TVdJn9RK8fAiAFDeGGyQx68RbYoIO_s5j6A4QLwJSXDFUoVk3w$>
:
> @@ -0,0 +1,50 @@
+## Known issues:
Can we remove this README?
------------------------------
In test_server_python/README.md
<https://urldefense.com/v3/__https://github.com/openai/openai-apps-sdk-examples/pull/38*discussion_r2432368389__;Iw!!G92We9drHetJ8EofZw!dXSFSek9zUhCsT9ee7u3ygCtyfm2VRN8oCMdCbaR6TVdJn9RK8fAiAFDeGGyQx68RbYoIO_s5j6A4QLwJSXDHWDX9pE$>
:
> @@ -0,0 +1,50 @@
+## Known issues:
+I haven't been able to get this to work yet in the 'test' widget.
+```
+ const gotoDoc = async () => {
+ // todo: this isn't working for me yet.
+ window.openai.openExternal({ "payload": "https://developers.openai.com/apps-sdk" });
+ };
+```
+
+## Static files are used from the /static path on the MCP server
+
+After running ```pnpm run build``` you need to copy all the test*.* files in the /assets directory to the /static folder under this server.
You shouldn't need to do that specifically for this example, just adding
the app to build-all is enough.
—
Reply to this email directly, view it on GitHub
<https://urldefense.com/v3/__https://github.com/openai/openai-apps-sdk-examples/pull/38*pullrequestreview-3340104017__;Iw!!G92We9drHetJ8EofZw!dXSFSek9zUhCsT9ee7u3ygCtyfm2VRN8oCMdCbaR6TVdJn9RK8fAiAFDeGGyQx68RbYoIO_s5j6A4QLwJSXDMVVOuz0$>,
or unsubscribe
<https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAKXL4IU4WI7AVTCAXWPKPT3XY45ZAVCNFSM6AAAAACI37RLJCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTGNBQGEYDIMBRG4__;!!G92We9drHetJ8EofZw!dXSFSek9zUhCsT9ee7u3ygCtyfm2VRN8oCMdCbaR6TVdJn9RK8fAiAFDeGGyQx68RbYoIO_s5j6A4QLwJSXDg8dyCIA$>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A simpler 'Test' widget and server - showing use of the window.openai function calls from the widget.