Skip to content

Accept NLX_MCP_URL in addition to NLX_APP_URL - #5

Open
dtolb wants to merge 3 commits into
mainfrom
dtolb-MCP-URL
Open

Accept NLX_MCP_URL in addition to NLX_APP_URL#5
dtolb wants to merge 3 commits into
mainfrom
dtolb-MCP-URL

Conversation

@dtolb

@dtolb dtolb commented Jun 10, 2025

Copy link
Copy Markdown

Solves #4

@dtolb
dtolb requested a review from a team as a code owner June 10, 2025 21:54
@dtolb

dtolb commented Jun 10, 2025

Copy link
Copy Markdown
Author

@sam-trost & @ndrppnc - small update to align UI and SDK

@peterszerzo

peterszerzo commented Jun 11, 2025

Copy link
Copy Markdown

Solves #4

@dtolb what do they each do, NLX_MCP_URL and NLX_APP_URL?

@dtolb

dtolb commented Jun 11, 2025

Copy link
Copy Markdown
Author

Solves #4

@dtolb what do they each do, NLX_MCP_URL and NLX_APP_URL?

NLX_MCP_URL is the proper name we should use. Supporting NLX_APP_URL for backwards compatibility

Screenshot 2025-06-11 at 09 49 42

@peterszerzo

peterszerzo commented Jun 11, 2025

Copy link
Copy Markdown

Solves #4

@dtolb what do they each do, NLX_MCP_URL and NLX_APP_URL?

NLX_MCP_URL is the proper name we should use. Supporting NLX_APP_URL for backwards compatibility

Screenshot 2025-06-11 at 09 49 42

The feature is not out, we should not care about backwards-compatibility at this stage. never mind it's the package, that makes sense then

Comment thread src/index.ts Outdated
Co-authored-by: Peter Szerzo <szerzo.peter@gmail.com>
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.

2 participants