Skip to content

feat(core-api): filter peers by version range#3465

Merged
faustbrian merged 3 commits intoArkEcosystem:2.6from
dated:feat/peers-version-range
Feb 13, 2020
Merged

feat(core-api): filter peers by version range#3465
faustbrian merged 3 commits intoArkEcosystem:2.6from
dated:feat/peers-version-range

Conversation

@dated
Copy link
Copy Markdown
Contributor

@dated dated commented Feb 11, 2020

Summary

Changes the peers API endpoint so that it can also accept a version range for retrieval of multiple versions at once.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@dated dated changed the base branch from master to develop February 11, 2020 21:28
@dated dated force-pushed the feat/peers-version-range branch from 07eb90b to 05d289d Compare February 11, 2020 22:20
@dated dated marked this pull request as ready for review February 12, 2020 07:03
@air1one
Copy link
Copy Markdown
Contributor

air1one commented Feb 13, 2020

hey @dated can you create a new PR for this against the 2.6 branch ? thanks !

@faustbrian faustbrian changed the base branch from develop to 2.6 February 13, 2020 08:09
@dated
Copy link
Copy Markdown
Contributor Author

dated commented Feb 13, 2020

thanks @faustbrian!

@ghost
Copy link
Copy Markdown

ghost commented Feb 13, 2020

A collaborator has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.

Thank you for your contribution!

@faustbrian faustbrian merged commit 1359ba2 into ArkEcosystem:2.6 Feb 13, 2020
@ghost
Copy link
Copy Markdown

ghost commented Feb 13, 2020

Your pull request has been merged but was not assigned a bounty tier. A maintainer will assign a bounty tier to this pull request in the next few days.

@ghost
Copy link
Copy Markdown

ghost commented Feb 13, 2020

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.

@dated dated deleted the feat/peers-version-range branch February 13, 2020 10:55
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.

3 participants