Giter Site home page Giter Site logo

Comments (9)

willpower232 avatar willpower232 commented on June 23, 2024

Thanks for your interest.

I don't think there is a way of specifying it currently but if you're able to make a PR to add in an optional array of extra fields to the get function which gets merged into the array which contains language then we can get that released for you

public function get(string $id, string $reportLang = 'en') : Company
{
return new Company($this->client, $this->client->get('companies/'.$id, ['language' => $reportLang]));

from creditsafe-connect.

DaanGeurts avatar DaanGeurts commented on June 23, 2024

Hi Will,

I found this PR #8
But it's not merged due to linting and formatting...

from creditsafe-connect.

willpower232 avatar willpower232 commented on June 23, 2024

Ah so it is, well they clearly have no intent on completing it.

The change I described above is only two lines worth of change so would be a much simpler and future proof solution if you have some time to work on it?

from creditsafe-connect.

DaanGeurts avatar DaanGeurts commented on June 23, 2024

Yes it could be a simple addition but there is more to it as you can read from the documentation. There is an extra endpoint to fetch all the available reason codes per country.
If you agree with a array merge of params and not adding the endpoint etc I'm more then happy to create a pull request

from creditsafe-connect.

willpower232 avatar willpower232 commented on June 23, 2024

Of course, if you want to grab and maybe improve src/Service/ReportCustomData.php from that PR as part of yours then that is probably a complete solution.

from creditsafe-connect.

DaanGeurts avatar DaanGeurts commented on June 23, 2024

Are you happy with the structure and code of the PR as I can copy that for my PR?

from creditsafe-connect.

willpower232 avatar willpower232 commented on June 23, 2024

if the linter is happy with you then it should be fine, that PR seemed to change a lot of code that wasn't related to the original issue and, whilst probably necessary at some point, really confused the issue 😅

from creditsafe-connect.

DaanGeurts avatar DaanGeurts commented on June 23, 2024

Hi Will,
i've created a new pull request based of the pull request mentioned here.
Let me know!

from creditsafe-connect.

willpower232 avatar willpower232 commented on June 23, 2024

Looks great to me, I've released it so feel free to close if you have no more issues on this topic

from creditsafe-connect.

Related Issues (5)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.