-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #4 from OneSignal/api
Dependency Update
- Loading branch information
Showing
69 changed files
with
11,472 additions
and
1,228 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
name: 🙋♂️ Ask a question | ||
description: Tell us what's on your mind | ||
title: "[question]: " | ||
labels: ["triage"] | ||
body: | ||
- type: markdown | ||
attributes: | ||
value: | | ||
Having issues integrating this API library? | ||
- type: textarea | ||
id: question | ||
attributes: | ||
label: How can we help? | ||
description: Specific question regarding integrating this API library. | ||
placeholder: How do I...? | ||
validations: | ||
required: true | ||
- type: checkboxes | ||
id: terms | ||
attributes: | ||
label: Code of Conduct | ||
description: By submitting this issue, you agree to follow our [Code of Conduct](https://github.com/OneSignal/api/blob/main/CONTRIBUTING.md) | ||
options: | ||
- label: I agree to follow this project's Code of Conduct | ||
required: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
name: 🪳 Bug report | ||
description: File a bug report | ||
title: "[Bug]: " | ||
labels: ["bug", "triage"] | ||
body: | ||
- type: markdown | ||
attributes: | ||
value: | | ||
Thanks for taking the time to fill out this bug report! | ||
- type: textarea | ||
id: what-happened | ||
attributes: | ||
label: What happened? | ||
description: Provide a thorough description of whats going on. | ||
placeholder: The latest version of the API library throws an exception when creating a notification targetting all Active Users. | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: reproduction-steps | ||
attributes: | ||
label: Steps to reproduce? | ||
description: Provide as much detail as posible to reproduce the issue. | ||
placeholder: | | ||
1. Install vX.Y.Z of dependency | ||
2. Run provided code snippet | ||
3. Note that the app crashes | ||
render: Markdown | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: what-are-expectations | ||
attributes: | ||
label: What did you expect to happen? | ||
description: Also tell us, what did you expect to happen? | ||
placeholder: I expected the API library to properly deserialize any response returned by OneSignal. | ||
validations: | ||
required: true | ||
- type: textarea | ||
id: logs | ||
attributes: | ||
label: Relevant log output | ||
description: Please copy and paste any relevant log output. This will be automatically formatted into code, so no need for backticks. | ||
render: Shell | ||
- type: checkboxes | ||
id: terms | ||
attributes: | ||
label: Code of Conduct | ||
description: By submitting this issue, you agree to follow our [Code of Conduct](https://github.com/OneSignal/api/blob/main/CONTRIBUTING.md) | ||
options: | ||
- label: I agree to follow this project's Code of Conduct | ||
required: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
name: 📣 General feedback | ||
description: Tell us what's on your mind | ||
title: "[Bug]: " | ||
labels: ["triage"] | ||
body: | ||
- type: markdown | ||
attributes: | ||
value: | | ||
Thanks for sharing your valuable feedback! | ||
- type: textarea | ||
id: feedback | ||
attributes: | ||
label: What's on your mind? | ||
description: Feedback regarding this API library. | ||
placeholder: Share your feedback... | ||
validations: | ||
required: true | ||
- type: checkboxes | ||
id: terms | ||
attributes: | ||
label: Code of Conduct | ||
description: By submitting this issue, you agree to follow our [Code of Conduct](https://github.com/OneSignal/api/blob/main/CONTRIBUTING.md) | ||
options: | ||
- label: I agree to follow this project's Code of Conduct | ||
required: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
<!-- START --> | ||
# READ AND DELETE THIS SECTION BEFORE SUBMITTING PR | ||
* **Fill out each _REQUIRED_ section** | ||
* **Fill out _OPTIONAL_ sections, remove section if it doesn't apply to your PR** | ||
* **Read and fill out each of the checklists below** | ||
* **Remove this section after reading** | ||
<!-- END --> | ||
|
||
# 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 | ||
- [ ] I have filled out all **REQUIRED** sections above | ||
- [ ] PR does one thing | ||
- If it is hard to explain how any codes changes are related to each other then it most likely needs to be more than one PR | ||
- [ ] Any Public API changes are explained in the PR details and conform to existing APIs | ||
|
||
## Testing | ||
- [ ] I have personally tested this on my device, or explained why that is not possible | ||
|
||
## Final pass | ||
- [ ] Code is as readable as possible. | ||
- Simplify with less code, followed by splitting up code into well named functions and variables, followed by adding comments to the code. | ||
- [ ] I have reviewed this PR myself, ensuring it meets each checklist item | ||
- WIP (Work In Progress) is ok, but explain what is still in progress and what you would like feedback on. Start the PR title with "WIP" to indicate this. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
## Golang | ||
1. Create a release tag | ||
2. `GOPROXY=proxy.golang.org go list -m github.com/OneSignal/onesignal-go-api@v<X.X.X>` |
Oops, something went wrong.