Skip to content

Conversation

magsyg
Copy link
Contributor

@magsyg magsyg commented Apr 22, 2020

Added EEA Sources which had an abundance of extra stations compared to the country's existing sources, these are:

  • Poland
  • Italy
  • Sweden

closes #677

@magsyg magsyg mentioned this pull request Apr 29, 2020
@espenairmine
Copy link
Contributor

Need to wait with this PR till the following PR's have been approved, as we need to update filtering for EEA for Italy:
#366
#720
#721

Copy link
Contributor

@sruti sruti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Following what I mentioned in #712 and #710, I would say Italy and Sweden are good to go, with a change to disable existing Italy adapters (change active to false).
Poland requires some further investigation, so create a separate PR for that?
Thanks!

Copy link
Contributor

@sruti sruti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@sruti sruti merged commit 0072931 into openaq:develop May 6, 2020
sruti added a commit that referenced this pull request May 6, 2020
Added EEA Sources for Sweden and Italy (#716)
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.

Several countries in EEA we are not pulling
3 participants