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

Add sample: Affiliate hook #37

Open
wants to merge 14 commits into
base: hooks-ssvm
Choose a base branch
from

Conversation

kromsten
Copy link

High Level Overview of Change

Adds an additional hook example project.

Context of Change

Extending a list of examples.

Type of Change

New example project that follows the same schema and architecture. The only difference is it uses a publis wss server instead of localhost as default.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Tests (You added tests for code that already exists, or your new feature included in this PR)
  • Documentation Updates
  • Release

@WietseWind WietseWind changed the title Hooks ssvm Add sample: Affiliate hook Feb 7, 2022
@RichardAH
Copy link
Member

Can you guys squash (rebase) these commits or squash them on another branch and re-open PR so it's just a single commit? Thanks!

kromsten added 2 commits March 8, 2022 12:56
removed reduntant param

usage instructions

added a cashback comission to the sender

minor refactoring

Update and rename readme.txt to README.md

Readme update

Updated usage scripts

Readme: fix for mention link

Revert "Readme: fix for mention link"

This reverts commit 173537f.

readme: mention github link

readme: markdown
@kromsten kromsten closed this Mar 8, 2022
@kromsten kromsten reopened this Mar 8, 2022
@kromsten kromsten closed this Mar 8, 2022
@kromsten kromsten reopened this Mar 8, 2022
@kromsten kromsten closed this Mar 8, 2022
@kromsten
Copy link
Author

kromsten commented Mar 8, 2022

Squashed

@kromsten kromsten reopened this Mar 8, 2022
@ihomp
Copy link

ihomp commented Mar 9, 2022

@RichardAH does it look better now, or would you prefer a new PR with one commit from a new branch :)

@Tomyleexrp

This comment was marked as abuse.

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.

4 participants