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

feat: standalone monaco API #1575

Merged
merged 3 commits into from
Jun 11, 2020
Merged

feat: standalone monaco API #1575

merged 3 commits into from
Jun 11, 2020

Conversation

acao
Copy link
Member

@acao acao commented Jun 10, 2020

  • replace monaco.languages.graphql with an api export object
  • combine api and defaults into one interface
  • get rid of .d.ts typings and module extension which caused issues for normal .ts typings
  • update documentation to reflect API changes
  • add setSchema()

@acao
Copy link
Member Author

acao commented Jun 10, 2020

heads up @Nishchit14 - changing this API so it's easier to use! thanks @rebornix for the advice on this. works much better this way. no more typescript mess @cshaver !

@acao acao force-pushed the feature/rewrite-improve-monaco-api branch 2 times, most recently from dd90464 to 084940b Compare June 10, 2020 10:00
- replace `monaco.languages.graphql` with an `api` export object
- combine api and defaults into one interface
- get rid of `.d.ts` typings and module extension which caused issues for normal .ts typings
- update documentation to reflect API changes
@acao acao force-pushed the feature/rewrite-improve-monaco-api branch from 084940b to 068a9bf Compare June 10, 2020 10:02
@acao acao force-pushed the feature/rewrite-improve-monaco-api branch from 1be45f2 to 7e307f4 Compare June 10, 2020 14:48
@acao acao changed the title feature: pull monaco API out and make it better feature: standalone monaco API Jun 10, 2020
@acao acao force-pushed the feature/rewrite-improve-monaco-api branch 3 times, most recently from d6d8b22 to a2c3b74 Compare June 11, 2020 14:26
@acao acao changed the title feature: standalone monaco API feat: standalone monaco API Jun 11, 2020
@acao acao force-pushed the feature/rewrite-improve-monaco-api branch from a2c3b74 to d2850d9 Compare June 11, 2020 14:51
@acao acao merged commit 954aa3d into master Jun 11, 2020
@acao acao deleted the feature/rewrite-improve-monaco-api branch June 11, 2020 15:00
@acao acao added this to the [email protected] milestone Jun 11, 2020
thenamankumar pushed a commit to thenamankumar/graphiql that referenced this pull request Aug 30, 2020
- replace monaco.languages.graphql with an api export object
- combine api and defaults into one interface
- get rid of .d.ts typings and module extension which caused issues for normal .ts typings
- update documentation to reflect API changes
- add setSchema()
- use `createData.languageConfig` for LSP config
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.

1 participant