Comments (9)
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
creditsafe-connect/src/Service/CompanyService.php
Lines 43 to 45 in 65a29f1
from creditsafe-connect.
Hi Will,
I found this PR #8
But it's not merged due to linting and formatting...
from creditsafe-connect.
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.
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.
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.
Are you happy with the structure and code of the PR as I can copy that for my PR?
from creditsafe-connect.
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.
Hi Will,
i've created a new pull request based of the pull request mentioned here.
Let me know!
from creditsafe-connect.
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
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from creditsafe-connect.