cardano-foundation / developer-portal

The Cardano Developer Portal
https://developers.cardano.org
MIT License
376 stars 906 forks source link

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

Open SupernaviX opened 4 weeks ago

SupernaviX commented 4 weeks ago

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 commented 4 weeks ago

@CarlosLopezDeLara this appears a consequence of https://github.com/cardano-foundation/developer-portal/pull/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 https://github.com/cardano-foundation/developer-portal/pull/1280.

CarlosLopezDeLara commented 4 weeks ago

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

CarlosLopezDeLara commented 4 weeks ago

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 commented 3 weeks ago

Thank you for the quick response!

SupernaviX commented 2 weeks ago

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