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

Remove duplicate feed for Donkey Republic in Rotterdam #689

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

richfab
Copy link
Contributor

@richfab richfab commented Oct 10, 2024

This PR removes a duplicate feed from @DonkeyRepublic in systems.csv for Rotterdam.

Indeed, the feed for the system with id donkey_rotterdam_den_haag contains exactly the same information as the feed for the system with id donkey_rt, minus some information. The first feed (with less information) should be removed to avoid duplicates.

Identical information in the following files for donkey_rt and donkey_rotterdam_den_haag:

  • station_information.json
  • station_status.json
  • system_hours.json
  • system_regions.json
  • gbfs_versions.json
  • vehicle_types.json

Extra information in the following files of donkey_rt:

  • system_information (license_url, purchase_url, url)
  • system_pricing_plans.json (url)
  • free_bike_status (one extra bike)

Thank you @merjakaj for raising this issue.

This PR removes a duplicate feed from @DonkeyRepublic [systems.csv](https://github.com/MobilityData/gbfs/blob/master/systems.csv) for Rotterdam.

Indeed, the feed for system id `donkey_rotterdam_den_haag` contains exactly the same information as the feed for system id `donkey_rt`, minus some information. The feed should be removed to avoid duplicates.

Identical information in the following files:
- station_information.json
- station_status.json
- system_hours.json
- system_regions.json
- gbfs_versions.json
- vehicle_types.json

Extra information in the following files of `donkey_rt`:
- system_information (license_url, purchase_url, url)
- system_pricing_plans.json (url)
- free_bike_status (one extra bike)

Thank you @merjakaj for raising this issue.
@richfab richfab marked this pull request as ready for review October 10, 2024 13:41
@richfab richfab merged commit 1a611c4 into master Oct 10, 2024
1 check passed
@richfab richfab deleted the remove-depublicate-feed-donkey-rotterdam branch October 10, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants