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

Remove references to PHP examples #44

Merged
merged 3 commits into from
Aug 30, 2018
Merged

Remove references to PHP examples #44

merged 3 commits into from
Aug 30, 2018

Conversation

basst85
Copy link
Contributor

@basst85 basst85 commented Jun 12, 2018

This PHP examples are no longer present on doc,bunq.com, and are also outdated.

@OGKevin
Copy link
Contributor

OGKevin commented Jun 14, 2018

@basst85 thanks for this, however i see that this repo is quite.. empty and missing the contribution guidelines 😅. Do you mind rebasing your commit and follow these little steps. This way we can have proper issue/commit tracking. https://raw.githubusercontent.com/bunq/sdk_php/develop/.github/PULL_REQUEST_TEMPLATE.md

Copy link
Contributor

@OGKevin OGKevin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Point the user to sdk source code for example.

DESCRIPTION.md Outdated
@@ -559,7 +559,7 @@ The calls you need to perform to set up a session from scratch are the following

Each call needs to be signed with your own private key. An Installation is used to tell the server about the public key of your key pair. The server uses this key to verify your subsequent calls.

Start by generating a 2048-bit RSA key pair. You can find out how to generate a key pair in our PHP coding examples.
Start by generating a 2048-bit RSA key pair.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of php coding examples, we can put something along the lines of you can find examples by looking at the source code of the sdk's located at github..

basst85 and others added 3 commits August 30, 2018 14:27
@OGKevin OGKevin merged commit fe80889 into bunq:develop Aug 30, 2018
@OGKevin
Copy link
Contributor

OGKevin commented Aug 30, 2018

@andrederoos

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.

2 participants