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

New projects api #35

Merged
merged 10 commits into from
Nov 21, 2022
Merged

New projects api #35

merged 10 commits into from
Nov 21, 2022

Conversation

ConanMishler
Copy link
Collaborator

@ConanMishler ConanMishler commented Nov 21, 2022

This PR closes the following issues:

#31
#27
#28
blocknetdx/exrproxy-env#206
blocknetdx/exrproxy-env#203
blocknetdx/exrproxy-env#204
blocknetdx/exrproxy-env#178

The behavior the Payment Processor will have after this PR is merged is described in the (freshly updated) Projects section of the API docs: https://api.blocknet.org/#projects-api-xquery-hydra

The SNode deployment docs (https://docs.blocknet.org/service-nodes/setup/#deploy-enterprise-xrouter-environment) have also been updated to reflect the changes made in this PR.

There are complementary PRs in the exrproxy and exrproxy-env repos of the same name which should be merged simultaneously with this one.

Note: The changes in this PR have only been about 75% tested. It really should be tested by others; not just me! That said, the easiest way to make it possible for others to test these changes is to merge this to master so a new docker image for the Payment Processor will be built and uploaded as the latest image on docker hub. This is obviously not good standard practice, but should be fine atm considering there are exactly ZERO people currently using the Payment Processor.

@atcsecure atcsecure self-assigned this Nov 21, 2022
@atcsecure atcsecure self-requested a review November 21, 2022 12:53
@shrnkld shrnkld merged commit c2b4087 into master Nov 21, 2022
@ConanMishler ConanMishler mentioned this pull request Dec 5, 2022
5 tasks
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