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

feat: add publishing actions and rename #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jonesus
Copy link

@Jonesus Jonesus commented May 28, 2022

closes #3

This PR would add automatic NPM publishing and GitHub artifact uploads whenever creating a new release (for example through the GitHub web interface).

Here we also update the package.json name-field to a directus-extension-prefix, so that this could be automatically installed to directus with npm install (directus/directus#12408). Maybe even a rename to directus-extension-dictionary-interface instead of this PRs directus-extension-dictionary could be used?

Steps needed for this to work:

  1. Create an NPM account
  2. Create an automation access token (https://docs.npmjs.com/creating-and-viewing-access-tokens)
  3. Add a secret to your repo named NPM_TOKEN (https://docs.github.com/en/codespaces/managing-codespaces-for-your-organization/managing-encrypted-secrets-for-your-repository-and-organization-for-codespaces#adding-secrets-for-a-repository)
  4. Merge this PR
  5. (Bump version?)
  6. Create a new release
  7. npm install directus-extension-dictionary 🎉

When trying this out (with npm package directus-extension-dictionary-test) I noticed that the interface doesn't seem to work like the GIF when setting it up (on [email protected] at least), but as I don't know how to fix that I hope this PR might motivate you or someone to fix it at some point, would be really great for generic i18n strings for my usecases :)

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.

Add automated releases
1 participant