Skip to content

Link to crisp button#12

Open
EffreinIBM wants to merge 1 commit intoCloud-Schematics:masterfrom
EffreinIBM:patch-1
Open

Link to crisp button#12
EffreinIBM wants to merge 1 commit intoCloud-Schematics:masterfrom
EffreinIBM:patch-1

Conversation

@EffreinIBM
Copy link
Copy Markdown

Take a look to see if I didn't mangle this. Trying to change link CTA to a crispy clear .svg button.

Take a look to see if I didn't mangle this. Trying to change link CTA to a crispy clear .svg button.
Comment thread README.md
You can use IBM Cloud Schematics to deploy this Terraform template in the cloud without having a local terraform configuration. Just use this link:

- [Deploy on IBM Cloud](https://cloud.ibm.com/schematics/workspaces/create?repository=https://github.com/Cloud-Schematics/openshift-dev-cluster&terraform_version=terraform_v0.12)
[![Deploy to Kubernetes on {{site.data.keyword.cloud_notm}}](../images/Deploy_to_Openshift.svg "Deploy to Kubernetes on {{site.data.keyword.cloud_notm}}")](https://cloud.ibm.com/schematics/workspaces/create?repository=https://github.com/Cloud-Schematics/openshift-dev-cluster&terraform_version=terraform_v0.12){: external}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

2 things I see here:

  1. This uses a relative URL for the SVG, this should probably be an absolute url. In the readme the image link is broken. If this is being rendered somewhere else, then it's probably broken there too.

    image

  2. The markdown doesn't look quite right. At the beginning there looks like an extra [!, there's a {{site.data.keyword.cloud_notm}} reference, and at the end there's an extra {: external} annotation. Those annotations don't get rendered here on github. Are they supposed to show up somewhere else? Those look consistent with IBM Cloud Docs, which has a separate preprocessing process that they user during publication.z

    image
    image

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