Skip to content

Clarify README backend URL documentation for deployment contexts#2578

Merged
piyalbasu merged 4 commits intomasterfrom
copilot/sub-pr-2566-again
Apr 23, 2026
Merged

Clarify README backend URL documentation for deployment contexts#2578
piyalbasu merged 4 commits intomasterfrom
copilot/sub-pr-2566-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 6, 2026

The README was updated to use placeholder backend URLs instead of production URLs, but the accompanying comment still referenced "production Freighter backend", creating confusion about the intended usage.

Changes:

  • Updated comment from "These URLs point to the production Freighter backend" to "These URLs should point to your deployment of Freighter backend"
  • Aligns documentation with the placeholder values your_backend_v1_prod_url_here and your_backend_v2_prod_url_here

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Keeeeeeeks and others added 2 commits January 30, 2026 14:07
Co-authored-by: Keeeeeeeks <16293980+Keeeeeeeks@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README to replace production URLs with placeholders Clarify README backend URL documentation for deployment contexts Feb 6, 2026
Copilot AI requested a review from Keeeeeeeks February 6, 2026 18:57
Copy link
Copy Markdown
Contributor

@Keeeeeeeks Keeeeeeeks left a comment

Choose a reason for hiding this comment

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

LGTM

@Keeeeeeeks Keeeeeeeks marked this pull request as ready for review February 11, 2026 14:07
Base automatically changed from Keeeeeeeks-patch-1 to master April 23, 2026 00:38
Copilot AI review requested due to automatic review settings April 23, 2026 00:50
@piyalbasu piyalbasu merged commit 31bdf19 into master Apr 23, 2026
6 checks passed
@piyalbasu piyalbasu deleted the copilot/sub-pr-2566-again branch April 23, 2026 00:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates root README environment-variable guidance to avoid implying the placeholder backend URLs are “production” endpoints, clarifying that they should be set to whichever Freighter backend deployment a developer is using.

Changes:

  • Adjusted README wording to reference a user’s own Freighter backend deployment rather than “production Freighter backend”.
  • Aligned the explanatory comment with the existing placeholder URL values.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
```

These URLs point to the production Freighter backend. For more details on
These URLs should point to your deployment of Freighter backend. For more details on
Copy link

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

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

Grammatically, this reads a bit awkwardly; consider adding the article so it’s clear you mean the project’s backend (e.g., “your deployment of the Freighter backend”).

Suggested change
These URLs should point to your deployment of Freighter backend. For more details on
These URLs should point to your deployment of the Freighter backend. For more details on

Copilot uses AI. Check for mistakes.
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.

4 participants