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

chore: remove validation for ts api docs #485

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Dhaiwat10
Copy link
Member

This PR removes the validation in place for the TS SDK API docs since they have now been isolated in a different project on a different URL (FuelLabs/fuels-ts#3237) and they are now only referenced from the 'main' TS SDK docs.

Nazeeh21
Nazeeh21 previously approved these changes Oct 25, 2024
@petertonysmith94 petertonysmith94 marked this pull request as draft October 28, 2024 11:09
@petertonysmith94
Copy link
Contributor

petertonysmith94 commented Oct 28, 2024

We're looking to approach this migration in a phased approach, so we converted this to a draft.

danielbate
danielbate previously approved these changes Nov 25, 2024
Copy link
Contributor

@petertonysmith94 petertonysmith94 left a comment

Choose a reason for hiding this comment

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

It may be nice to keep the API link that opens a new window with the api-docs. I approve of these changes.

image

@Dhaiwat10
Copy link
Member Author

@petertonysmith94 not really sure how to do that. @FuelLabs/devrel any ideas? 👀

@danielbate
Copy link
Contributor

@petertonysmith94 @Dhaiwat10 I have included the link on the TS docs home page. We'd probably have to include it in the links json file but that'll probably break some kind of validation. IMO it should not block this work.

@danielbate
Copy link
Contributor

@calldelegation @rishabhkeshan please let us know when you've had a chance to check this PR

@petertonysmith94
Copy link
Contributor

https://vercel.com/fuel-labs/docs-hub/22k6WPftQk4vYpSDCd49NkuGpgzM#L361

ERROR GETTING EXAMPLE CODE: Error: ENOENT: no such file or directory, open '/vercel/path0/docs/nightly/fuels-ts/apps/docs/nightly/fuels-ts/apps/docs/src/guide/contracts/snippets/cost-estimation.ts'

It looks like the final error is related to the relative path snippets introduced in #505 (which worked as expected locally when I tested it).

I believe the path referenced:
'/vercel/path0/docs/nightly/fuels-ts/apps/docs/nightly/fuels-ts/apps/docs/src/guide/contracts/snippets/cost-estimation.ts'

Should be:
'/vercel/path0/docs/nightly/fuels-ts/apps/docs/src/guide/contracts/snippets/cost-estimation.ts'

@calldelegation do you know what might be happening here?

@calldelegation
Copy link
Contributor

@petertonysmith94 you are correct. @Dhaiwat10 and I were able to deduce down to the same issue yesterday which is the extra docs/nightly/fuels-ts/apps you are referring to.

Though it seems like an easy fix to simple trim the URL I am not sure on the top of my head but I can help investigate as soon as I have completed my higher priority tasks.

@petertonysmith94
Copy link
Contributor

@petertonysmith94 you are correct. @Dhaiwat10 and I were able to deduce down to the same issue yesterday which is the extra docs/nightly/fuels-ts/apps you are referring to.

Though it seems like an easy fix to simple trim the URL I am not sure on the top of my head but I can help investigate as soon as I have completed my higher priority tasks.

No rush - thanks @calldelegation 🙌🏼

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.

6 participants