Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Fix link on README.md - 1.9 #477

Merged
merged 1 commit into from
Mar 11, 2020
Merged

Conversation

lparisc
Copy link
Contributor

@lparisc lparisc commented Mar 10, 2020

Change Description

Parallel PR #476
Resolves #475

Deployment Changes

  • Deployment Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@lparisc lparisc self-assigned this Mar 10, 2020
@lparisc lparisc requested a review from iamveritas March 10, 2020 03:36
@lparisc lparisc added the documentation Requires attention of documentation team. e.g. documentation is expected to be generated. label Mar 10, 2020
To build the contracts follow the instructions in [`Build and deploy` section](./docs/02_build-and-deploy.md).
## Build

To build the contracts follow the instructions in [Build and deploy](https://developers.eos.io/manuals/eosio.contracts/latest/build-and-deploy) section.
Copy link
Contributor

@iamveritas iamveritas Mar 10, 2020

Choose a reason for hiding this comment

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

FYI: this should be
https://developers.eos.io/manuals/eosio.contracts/v1.9/build-and-deploy, it should not contain latest because this fix is specific for v1.9 version, and as it is now it will always point to latest, and we do not want that when let's say v2.0 will be deployed we want this v1.9 page to point to the corresponding v1.9 build and deploy procedure; however because of this current issue we can not use the correct link.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed - it is what it is right now

@lparisc lparisc merged commit f1db063 into release/1.9.x Mar 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Requires attention of documentation team. e.g. documentation is expected to be generated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants