Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update README.md fix click to deploy link #24

Closed
wants to merge 2 commits into from

Conversation

sc-zenokerr
Copy link
Contributor

No description provided.

Change domain back to my.scalingo.com fix query string
@@ -15,7 +15,7 @@ And that's it!

## Deploy via One-Click

[![Deploy to Scalingo](https://cdn.scalingo.com/deploy/button.svg)](https://my.scalingo.com/deploy)
[![Deploy to Scalingo](https://cdn.scalingo.com/deploy/button.svg)](https://my.scalingo.com/deploy?source=https://github.com/Scalingo/sample-node-express)
Copy link
Member

Choose a reason for hiding this comment

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

The source argument is not mandatory. It should work without it. Did you faced any issue using the one-click deploy here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, unfortunately the link does not work without the source parameter.

@sc-zenokerr
Copy link
Contributor Author

Superseded by a later change. No longer necessary

@sc-zenokerr sc-zenokerr closed this Oct 7, 2024
auto-merge was automatically disabled October 7, 2024 07:14

Pull request was closed

@sc-zenokerr sc-zenokerr deleted the sc-zenokerr-patch-1 branch October 7, 2024 07:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants