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

[providers] ajout naofood Nantes #18

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

arthurlutz
Copy link

No description provided.

@taratatach
Copy link
Collaborator

Hi @arthurlutz, thank you for your contribution ! 😃

I've ran the src/providers.js script and found out there are other new providers which could be included as well.
However the provider ids shouldn't be changed and the script does not take into account existing provider ids so we can't use its output as is.

Would you be willing to make the necessary changes to:
a. be able to fetch the list of providers from coopcycle.org as is already done
b. keep untouched the existing provider/providerId mapping
c. make sure providers are still displayed in alphabetical order to the user ?

Otherwise I'll do it myself but I can't guarantee when it'll be available.

Again, thank you for contributing and raising my awareness around in the addition of new providers 😃

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.

None yet

2 participants