This repository has been archived by the owner on Apr 27, 2022. It is now read-only.

Improvement suggestions #1

Closed

5 tasks done

Atinux opened this issue Apr 26, 2022 · 1 comment

Contributor

Atinux commented Apr 26, 2022 •

edited by Intevel

Update README to mention that they can simply add LOGSNAG_API_TOKEN environment in .env instead of setting in nuxt.config
Is the API token for log snag public? 🤔
Is it necessary? https://github.com/Intevel/nuxt-logsnag/blob/master/src/runtime/plugin.ts#L4
I would rename it to publish on https://github.com/Intevel/nuxt-logsnag/blob/master/src/runtime/composables/useLogSnag.ts#L26, like the npm module: https://www.npmjs.com/package/logsnag
Rename #app to #imports on https://github.com/Intevel/nuxt-logsnag/blob/master/src/runtime/composables/useLogSnag.ts#L2

The text was updated successfully, but these errors were encountered:

Atinux mentioned this issue

feat: added nuxt-logsnag nuxt/modules#358

Closed

Owner

Intevel commented Apr 26, 2022

Hey @Atinux, thanks.
Improved your suggestions in the newest release. 💚

Intevel closed this as completed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.