Skip to content

Extract docs #70

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

Merged
merged 36 commits into from
May 5, 2021
Merged

Extract docs #70

merged 36 commits into from
May 5, 2021

Conversation

tmlayton
Copy link
Contributor

@tmlayton tmlayton commented Apr 5, 2021

Todo

  • Fix squished description column
  • Update output paths to match the new taxonomies
  • Replace Github relative links with the Shopify.dev equivalent
  • Fix call stack error with View component
  • Fix Responsive props - resolving as an "UndocumentedType", maybe because it is an interface that is part of a union
  • Preserve unique locals (e.g. type Size in BlockSpacing, Text, and Layout are all different). Currently docs for Layout are incorrect and listing the string literal values for BlockSpacing.
  • Add generic arguments to interface titles
  • Generate both READMEs and shopify.dev docs
  • Parse extends - in progress
  • Handle more complex types like generics or literal interfaces (e.g. Text role)
  • Parse tsdoc tags (e.g. @default, etc) - Parse and render tsdoc tags in PropertySignature nodes #73
  • Fix void return for functions
  • Parse /examples/name.example.[ts|*] files
  • Parse /content folder to additonal markdown content
  • Use markdown tables OR properly parse basic markdown in table content (e.g. markdown links)
  • Fix overflow formatting on table
  • Metadata file?
  • Publish as versions?
  • Refactor all of this into composable parts to share with Argo Admin 😂

Content todo

  • Write examples for vanilla JS and React
  • Pull markdown formatting out and place in content files (e.g. CalloutBanner "Guidelines")
  • Update all doc comments to inline content. Flatten any lists or move to /content markdown files.
  • Move any doc comments to interfaces. Comments on type’s used in interfaces are not parsed.

@iainmcampbell iainmcampbell force-pushed the extract-docs branch 2 times, most recently from 823b1cb to 2b02b09 Compare April 28, 2021 20:53
@iainmcampbell
Copy link
Contributor

CI is failing on what appears to be a lack of dom types (ReactiveX/rxjs#1986 (comment))

…however, we already include the dom library in our compilerOptions in tsconfig.base.json 🤔

@tmlayton
Copy link
Contributor Author

tmlayton commented Apr 29, 2021

It looks like the TS config for the argo-checkout package is being used, which appears to overwrite the array of lib's when using a base config instead of merging them. When I swap WebWorker with dom the error goes away but I get another error related to the global declaration for shopify, it’s also not the right thing to do. I tried using both the web worker and dom libs but it throws a bunch of conflict errors.

Update: looks like this is a known error with projects that want to use both the web worker and dom libraries and a few workarounds here microsoft/TypeScript#20595

@iainmcampbell iainmcampbell marked this pull request as ready for review May 3, 2021 19:53
@iainmcampbell
Copy link
Contributor

I want to merge this in cause it's becoming a pain to rebase. It's also modular enough that Admin and Checkout can build on this separately.

The scripts are still WIP but if you wanna test them, make sure you have shopify-dev cloned and run yarn docs:admin or yarn docs:checkout.

@tmlayton
Copy link
Contributor Author

tmlayton commented May 4, 2021

Not sure how I feel about the skipLibCheck in the base config, we should be able to instead move the scripts to their own area with their own tsconfig which includes the dom lib.

Otherwise I’m fine with merging. If you do so I just ask that you move the outstanding todos to their own issues.

@iainmcampbell
Copy link
Contributor

@tmlayton tried that, can't seem to get it to work, see last commit

Copy link
Member

@vividviolet vividviolet left a comment

Choose a reason for hiding this comment

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

I couldn't get very far for testing unfortunately


export {extensionPoints, components, gettingStarted};

export function renderForShopifyDev(paths: Paths) {
Copy link
Member

Choose a reason for hiding this comment

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

Can we clone shopify-dev if it doesn't already exist? I'm getting this error when running the command without the dev repo cloned:

node:internal/fs/utils:343
    throw err;
    ^

Error: ENOENT: no such file or directory, mkdir '/Users/trishta/src/github.com/Shopify/shopify-dev/content/docs/checkout-extensions'
    at Object.mkdirSync (node:fs:1038:3)
    at gettingStarted (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/shopify-dev-renderer/getting-started.ts:15:8)
    at renderForShopifyDev (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/shopify-dev-renderer/index.ts:12:3)
    at Object.<anonymous> (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/generate-docs-checkout.ts:13:1)
    at Module._compile (node:internal/modules/cjs/loader:1108:14)
    at Module._compile (/Users/trishta/src/github.com/Shopify/argo-checkout/node_modules/pirates/lib/index.js:99:24)
    at Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
    at Object.newLoader [as .ts] (/Users/trishta/src/github.com/Shopify/argo-checkout/node_modules/pirates/lib/index.js:104:7)
    at Module.load (node:internal/modules/cjs/loader:988:32)
    at Function.Module._load (node:internal/modules/cjs/loader:828:14) {
  errno: -2,
  syscall: 'mkdir',
  code: 'ENOENT',
  path: '/Users/trishta/src/github.com/Shopify/shopify-dev/content/docs/checkout-extensions'

Copy link
Member

Choose a reason for hiding this comment

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

Second error after cloning shopify-dev:

node:fs:506
  handleErrorFromBinding(ctx);
  ^

Error: ENOENT: no such file or directory, open '/Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts'
    at Object.openSync (node:fs:506:3)
    at readFileSync (node:fs:402:35)
    at extractModule (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/utilities/dependency-graph.ts:134:18)
    at process (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/utilities/dependency-graph.ts:113:30)
    at take (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/utilities/dependency-graph.ts:90:9)
    at /Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/utilities/dependency-graph.ts:81:5
    at new Promise (<anonymous>)
    at createDependencyGraph (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/utilities/dependency-graph.ts:75:9)
    at extensionPoints (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/shopify-dev-renderer/extension-points.ts:18:23)
    at renderForShopifyDev (/Users/trishta/src/github.com/Shopify/argo-checkout/scripts/typedoc/shopify-dev-renderer/index.ts:10:3) {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts'
}

Copy link
Member

Choose a reason for hiding this comment

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

More errors after cloning checkout-web

Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllowedComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `RemoteRoot` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Partial` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.

Copy link
Contributor

Choose a reason for hiding this comment

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

hm, you might need to yarn install in checkout-web?

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we can/should clone repos automatically. i'll write a readme with these prereq steps.

Copy link
Member

Choose a reason for hiding this comment

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

I'm still seeing the type errors, are these expected?

TSTypeQuery is unhandled.
ArrowFunctionExpression is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSConditionalType is unhandled.
TSConditionalType is unhandled.
TSConditionalType is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllowedComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `RemoteRoot` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Partial` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

this script is still WIP! I will need to handle some of these for the extension points, others we can safely ignore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, this is to let us know where things are likely broken or not getting documented—both for typescript types and user defined types. Any "SomeType is unhandled" is a typescript feature we are using but the script does not yet support.

The unresolved errors are mostly for types imported from an external package that the script does not have access, too.

Copy link
Member

@vividviolet vividviolet left a comment

Choose a reason for hiding this comment

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

I finally got the scripts working but due to local issues I can't actually see the docs on shopify-dev so I couldn't fully top hat. The files look great though aside from the warnings

JavaScript: './packages/argo-admin',
React: './packages/argo-admin-react',
},
outputRoot: '../shopify-dev/content/tools/argo-admin',
Copy link
Member

Choose a reason for hiding this comment

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

Should we not put these docs in the same place as Checkout like docs/admin-extensions?

Copy link
Contributor

Choose a reason for hiding this comment

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

probably yes, haven't finalized this yet. not concerned about this right now.


export {extensionPoints, components, gettingStarted};

export function renderForShopifyDev(paths: Paths) {
Copy link
Member

Choose a reason for hiding this comment

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

I'm still seeing the type errors, are these expected?

TSTypeQuery is unhandled.
ArrowFunctionExpression is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSIndexedAccessType is unhandled.
TSConditionalType is unhandled.
TSConditionalType is unhandled.
TSConditionalType is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTypeLiteral is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
TSTupleType is unhandled.
Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Record` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `StatefulRemoteSubscribable` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Promise` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `AllowedComponents` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `RemoteRoot` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Input` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.
Can’t resolve export type `Partial` in /Users/trishta/src/github.com/Shopify/checkout-web/packages/argo-checkout/src/index.ts. Maybe it’s not exported from the component index or imported from a remote package.

@iainmcampbell
Copy link
Contributor

🎉🎉🎉
Screen Shot 2021-05-05 at 3 26 21 PM

Copy link
Member

@vividviolet vividviolet left a comment

Choose a reason for hiding this comment

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

Works great 👍

@iainmcampbell iainmcampbell merged commit 505f7a6 into main May 5, 2021
@iainmcampbell iainmcampbell deleted the extract-docs branch May 5, 2021 20:05
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.

3 participants