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

docs: convert examples to typescript #3766

Merged
merged 7 commits into from
Aug 19, 2020

Conversation

wzhudev
Copy link
Contributor

@wzhudev wzhudev commented Jul 6, 2020

ts-slate

Is this adding or improving a feature or fixing a bug?

Not relevant.

What's the new behavior?

Convert all example files to TypeScript and update next.js to 9.4.0.

How does this change work?

Have you checked that...?

  • The new code matches the existing patterns and styles.
  • The tests pass with yarn test.
  • The linter passes with yarn lint. (Fix errors with yarn fix.)
  • The relevant examples still work. (Run examples with yarn start.)

Does this fix any issues or need any specific reviewers?

Fixes: #3713
Reviewers: @CameronAckermanSEL

@wzhudev wzhudev marked this pull request as ready for review July 6, 2020 09:01
@wzhudev wzhudev marked this pull request as draft July 6, 2020 09:35
@wzhudev
Copy link
Contributor Author

wzhudev commented Jul 7, 2020

What I've changed:

  • Update Next.js to 9.4.0
  • Convert files under site to TypeScript
  • Change path mechanism using getStaticPaths

It seems that exportPathMap breaks in production mode after updating to Next.js 9.4.0. I am working on that.

@wzhudev wzhudev marked this pull request as ready for review July 7, 2020 02:13
@wzhudev
Copy link
Contributor Author

wzhudev commented Aug 10, 2020

Ping @CameronAckermanSEL.

@dustinsenos
Copy link

Curious why you’re converting, rather than not just adding TypeScript docs?

@cameracker
Copy link
Collaborator

Thank you so much for doing all of this work. I'm sorry this took so long for me to get to.

@cameracker cameracker merged commit 912d4b7 into ianstormtaylor:master Aug 19, 2020
@wzhudev
Copy link
Contributor Author

wzhudev commented Aug 31, 2020

Curious why you’re converting, rather than not just adding TypeScript docs?

I saw this converting thing days ago in the issue list. :P

lukesmurray pushed a commit to lukesmurray/slate that referenced this pull request Feb 5, 2021
* docs: convert examples to typescript

* docs: convert remaining examples

* docs: update next.js

* ci: fix lint

* docs: fix next.js path

* docs: cleanup

* update

Co-authored-by: wendellhu <wendellhu@tencent.com>
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.

Convert examples to typescript
3 participants