feat: Update Polygon to Massive due to MCP rename#583
Open
SatLibRe wants to merge 2 commits intoed-donner:mainfrom
Open
feat: Update Polygon to Massive due to MCP rename#583SatLibRe wants to merge 2 commits intoed-donner:mainfrom
SatLibRe wants to merge 2 commits intoed-donner:mainfrom
Conversation
Owner
|
I've just run the code precisely as it is in the repo, with the original Polygon MCP server name, and the polygon key, and it ran fine and returned Apple's stock price. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR updates all references from Polygon.io to Massive following their recent rebrand in
3_lab3of the MCP section.While a rebrand alone would not normally require a full update, Massive has also renamed and updated their MCP repository, along with related server configuration and environment variables. Specifically, the MCP server is now sourced from:
uvx --from git+https://github.com/massive-com/mcp_massive@v0.6.0 mcp_massiveWhen I tried to use the old setup, it failed. This changes fixes that.
This PR addresses those changes by:
Updating the MCP server to the latest stable version from Massive
Revising references to Polygon.io in
3_lab3of the MCP section of the course to reflect the new namingTo avoid introducing breaking changes,
polygon-api-client>=1.14.5has intentionally been left in the .toml file.Note: This change will require either a re-recording or a short supplemental video to explain the updates and keep the course content aligned with the code.