-
-
Notifications
You must be signed in to change notification settings - Fork 633
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
Added new about the developer section to the app
method
#693
Conversation
* added about developer section data * improved structure of the about developer data * simplified address
Let's call it devInfo, and update the readme please and the tests accordingly, please. |
Also I don't think this should be a separate object (I mean it would be best to have all dev fields into a separate object, but we're already using separate developerId, developerEmail, etc, and I prefer not to introduce a breaking change). You should use developerLegalName or developerCompany or something similar to avoid confusion, Likewise check whether the email in the about section is different from the email we already have and rename if necessary. |
Ok I will remove them from the object. I have noticed in some apps that the email can be different between this new section and the support email in the about section. |
@facundoolano On the apps that are used on the README and the tests, I see that the |
Code looks good, the only thing is that for the test, you should try to choose an app that's less likely to be removed to reduce the chances of breaking the test down the line. Did you chose this developer for some particular reason? |
No you are right. I will find an app that has the information and has less chance of being removed. |
@facundoolano I updated the app with soundcloud which should be more solid as a test |
Hello,
I wanted to add the new "About the developer" section which I think will soon be mandatory to all apps. (screenshot as reference)
I split it in its own object so its better organised. Let me know if you think that is okay.
If possible let me know what else I need to do for this PR to add this feature in the scraping.
I saw the tests are failing. I will add this change to the tests as well.