Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove obsolete "build transactions" guide (or link to the new one) #1313

Open
SupernaviX opened this issue Aug 30, 2024 · 5 comments
Open
Labels
documentation Improvements or additions to documentation

Comments

@SupernaviX
Copy link

Hello! I'm a new developer in the Cardano ecosystem, and I just spent a few days trying to submit my first transaction. I was following this "Create Simple Transaction" guide mostly word-for-word, and it recommends using cardano-cli transaction build-raw to construct transactions (and cardano-cli transaction calculate-min-fee to compute the fees yourself).

After working through my issues, I learned that this guide was obsolete for a few reasons, and that I should have been following this "Cardano CLI Simple Transactions" guide instead. The old guide is the only one that shows up for me on Google, and I didn't even notice the new one until I was writing this very issue.

I think the old guide should get taken down so other people don't make the same mistake I made.

@rphair
Copy link
Collaborator

rphair commented Aug 30, 2024

@CarlosLopezDeLara this appears a consequence of #1287 - would you like to remove the old page & redirect it to your new one, and/or merge the content somehow? (though I feel sure you've already accommodated everything useful from the old page).

You could follow the redirect syntax in docusaurus.config.js last used extensively in #1280.

@rphair rphair added the documentation Improvements or additions to documentation label Aug 30, 2024
@CarlosLopezDeLara
Copy link
Collaborator

I'll take care of it. Thanks for the report @SupernaviX

@CarlosLopezDeLara
Copy link
Collaborator

Ok, I removed the file in question to see what links would break:

Exhaustive list of all broken links found:

- On source page path = /docs/get-started/secure-workflow:
   -> linking to /docs/get-started/create-simple-transaction
   -> linking to /docs/get-started/create-simple-transaction#determine-the-ttl-time-to-live-for-the-transaction (resolved as: /docs/get-started/create-simple-transaction)
   -> linking to /docs/get-started/create-simple-transaction#check-the-balances (resolved as: /docs/get-started/create-simple-transaction)

After reading the article that links to this, I think it will need some refactoring as well. This to say that this may take a bit longer than expected. I'll start working on it and hopefully have a PR for Monday - Tuesday.

@SupernaviX
Copy link
Author

Thank you for the quick response!

@SupernaviX
Copy link
Author

If removing the page is difficult, could the old page link to the new one at the top as a short-term workaround?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants