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

TypeScript definition fixes #1

Open
wants to merge 1 commit into
base: vue-3
Choose a base branch
from

Conversation

ElteHupkes
Copy link

I'm attempting to convert a project using Buefy to Vue3 using this fork, so far very successfully (thanks!). It's also using TypeScript, and I realized those definitions have probably not been touched. Here's a few basic ones, which at least make sure app.use(Buefy) isn't an error, and programmatic dialogs don't suggest onConfirm / onCancel anymore. I want to look at the other definitions for programmatic access later.

- onConfirm / onCancel replaced with `confirmCallback` / `cancelCallback`
- The main Buefy `install` function now adheres to the Vue3 Plugin_2 type
kikuomax added a commit that referenced this pull request Sep 1, 2023
* chore: upgrade jest-serializer-vue

- Bumps jest-serializer-vue from 0.3.0 to 3.1.0 because version 0.3.0
  shall not work with V2 of `@vue/test-utils`.

  See: eddyerburgh/jest-serializer-vue#56

* chore: install jest-dom

- Installs `@testing-library/jest-dom` to enable `toBeEmptyDOMElement`
  assertion.

* test(lib): polyfill window.crypto

- Polyfills `window.crypto` in the test environment to provide
  `window.crypto.getRandomValues`. It is defined in a newly introduced
  file `jest-setup.js` that runs before every test case.

issue ntohq#1 (comment)

* chore(lib): uninstall sinon

- Uninstalls `sinon` because no tests directly depend on it.

issue ntohq#1

* chore: resync package name and version

- Name → @ntohq/buefy-next
- Version → 0.1.1
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