Skip to content

Applying to add new php library #1619

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

Merged
merged 1 commit into from
Mar 26, 2022
Merged

Applying to add new php library #1619

merged 1 commit into from
Mar 26, 2022

Conversation

elb98rm
Copy link
Contributor

@elb98rm elb98rm commented Mar 23, 2022

Applying to add new php library: https://github.com/floor9design-ltd/json-api-formatter

Info:

  • 100% unit tests
  • CI implemented
  • 6 use months in production environments in multiple projects
  • actively maintained

Applying to add new php library: https://github.com/floor9design-ltd/test-data-generator

Info: 
* 100% unit tests
* CI implemented
* 6 use months in production environments in multiple projects
* actively maintained
@jelhan
Copy link
Contributor

jelhan commented Mar 26, 2022

Thanks a lot for opening this PR. I'm sure it's a great library but how is it related to JSON:API specification? I skimmed over the readme and didn't found the specification to be mentioned at all.

@elb98rm
Copy link
Contributor Author

elb98rm commented Mar 26, 2022

Not sure I understand the feedback. Can you help explain?

Just to be clear: It's not about the specification - this is a library allowing people to import/export their data to/from JSON API.

Note that many others in the list do the same, for example:

If you want to see if I'm referencing the JSON API spec, it's directly linked (along with a validator) in the testing area.

@jelhan
Copy link
Contributor

jelhan commented Mar 26, 2022

this is a library allowing people to import/export their data to/from JSON API.

Are you sure that you included the correct link in the post? It links to test-data-generator, which is described as the following:

A class for generating simple data, ideal for testing/seeding or other random code requirements.

Would be great if you could update the title of this pull request as well to reflect what is it doing.

@elb98rm
Copy link
Contributor Author

elb98rm commented Mar 26, 2022

D'oh

That's a DIFFERENT repo I'm working on - must have been a copy paste issue!
Updating this now.

@elb98rm
Copy link
Contributor Author

elb98rm commented Mar 26, 2022

Corrected repo link in request above. Note that the repo link in the code was correct.

@elb98rm elb98rm changed the title Update index.md Applying to add new php library Mar 26, 2022
Copy link
Contributor

@jelhan jelhan left a comment

Choose a reason for hiding this comment

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

Thanks a lot for fixing the link in PR description. To be honest I haven't had a look in the commit before.

Looks good to me. 👍

I don't have merge permissions myself. Need to wait for @dgeb to give it a final review and merge.

@dgeb
Copy link
Member

dgeb commented Mar 26, 2022

Looks good to me as well. Thanks @elb98rm for the library and @jelhan for the review!

@dgeb dgeb merged commit d7de6a7 into json-api:gh-pages Mar 26, 2022
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy