Skip to content
This repository has been archived by the owner on May 19, 2022. It is now read-only.

next version first steps #60

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

next version first steps #60

wants to merge 15 commits into from

Conversation

claudiocro
Copy link
Member

No description provided.

@coveralls
Copy link

coveralls commented Jan 28, 2019

Coverage Status

Coverage decreased (-0.6%) to 99.032% when pulling 28b2510 on features/next into 6c4fac7 on master.

BREAKING CHANGE: Refactor and cleanup the whole project.

Removed the options language, we use the i18next implemantation to controll the language.

```js

// Before
{ path: 'helloPerson', language: 'de', args: { name: this.name } }

// After
{ path: 'helloPerson', args: { name: this.name, lng: 'de' } }
```

The option `getComponentNamespace` has been removed in favor of `componentNamespace`.

```js
// Before
const vueI18Next = new VueI18Next(i18next, {
  getComponentNamespace: () => ({ namespace: "comp-ns", loadNamespace: true })
});

// After
const vueI18Next = new VueI18Next(i18next, {
  componentNamespace: "comp-ns"
});
```
Change the way we load namespaces.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants