The Vue 3 injection Symbol should be in a separate file and exported

[idc77]

The Vue 3 injection Symbol should be in a separate file and exported

[baltom]

@idc77 Had to revert your commit/pr as it dosen't build see: https://rollupjs.org/configuration-options/#output-exports

Also note that it KeycloakSymbol is exported here https://github.com/dsb-norge/vue-keycloak-js/blob/main/src/index.ts#L42

import VueKeyCloak from '@dsb-norge/vue-keycloak-js'
const symbol = VueKeyCloak.KeycloakSymbol

[idc77]

@chrisnruud @baltom @josteitv
You probably don't want to be pinged but this is an easy change which would make away with the getCurrentInstance() requirement.

[idc77]

Ok. After MUCH experimentation the simplest solution is to just add export to the KeycloakSymbol declaration.

old

const KeycloakSymbol = Symbol('keycloak')

new

export const KeycloakSymbol = Symbol('keycloak')

And with this one can just

import {KeycloakSymbol} from "@dsb-norge/vue-keycloak-js";

// ... setup() 
    const kc = inject(KeycloakSymbol)

Please just make this small change.

Meanwhile I've copy/pasted the change to
https://www.npmjs.com/package/dalu-vue3-keycloak
https://code.icod.de/dalu/vue3-keycloak

[chrisnruud]

Thank you for the ping. We will try to take a look at the earliest convenience. Sorry for not responding in a timely manner :)

[idc77]

Meanwhile 2 weeks have passed and nothing happened. Writing 7 characters is very time consuming.
Come on already.

[chrisnruud]

@idc77 thank you for the PR!

[idc77]

wow, so much trouble over nothing.
Sorry to have bothered you.