OneSignal / onesignal-node-api

OneSignal Node Client
Other
28 stars 12 forks source link

Merge pull request #33 from OneSignal/test_cd_npm_deploy #34

Closed kesheshyan closed 1 year ago

kesheshyan commented 1 year ago

Updated NPM publish action to trigger on release tag added.

READ AND DELETE THIS SECTION BEFORE SUBMITTING PR

Description

One Line Summary

REQUIRED - Very short description that summaries the changes in this PR.

Details

Motivation

REQUIRED - Why is this code change being made? Or what is the goal of this PR? Examples: Fixes a specific bug, provides additional logging to debug future issues, feature to allow X.

Scope

RECOMMEND - OPTIONAL - What is intended to be effected. What is known not to change. Example: Notifications are grouped when parameter X is set, not enabled by default.

OPTIONAL - Other

OPTIONAL - Feel free to add any other sections or sub-sections that can explain your PR better.

Testing

Manual testing

REQUIRED - Explain what scenarios were tested and the environment.

Checklist

Overview

Testing

Final pass