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

test: lifespan test #379

Merged
merged 26 commits into from
Mar 4, 2021
Merged

test: lifespan test #379

merged 26 commits into from
Mar 4, 2021

Conversation

posva
Copy link
Member

@posva posva commented Mar 3, 2021

Fix #370

This is a big refactor that aligns v1 with v2 and brings many improvements

posva added 16 commits March 3, 2021 15:27
Fix #370

BREAKING_CHANGE: It's now necessary to create a pinia instance and
install it:
  ```js
  import { createPinia, PiniaPlugin } from 'pinia'

  const pinia = createPinia()
  Vue.use(PiniaPlugin)

  new Vue({
    el: '#app',
    pinia,
    // ...
  })
  ```
  The `pinia` instance can be passed to `useStore(pinia)` when called
  outside of a `setup()` function. Check the SSR section of the docs for
  more details.

BREAKING_CHANGE: `setActiveReq()` and `getActiveReq()` have been
replaced with `setActivePinia()` and `getActivePinia()` respectively.
`setActivePinia()` can only be passed a `pinia` instance created with
`createPinia()`.

BREAKING_CHANGE: Since req as a parameter was replacetd with `pinia`,
`getRootState` is no longer necessary. Replace it with
`pinia.state.value` to **read and write** the root state`.

BREAKING_CHANGE: `PiniaSsr` is no longer necessary and has been removed.
@posva posva merged commit 08bd752 into v1 Mar 4, 2021
@posva posva deleted the v1-prepare-for-alpha branch March 4, 2021 15:43
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.

None yet

1 participant