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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add csv displayer, table and map components and add automatic tests and code coverage #37

Open
wants to merge 63 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@DavidePastore
Member

DavidePastore commented Oct 8, 2017

New components

  • Csv Displayer (related to #3 and #6);
  • Table (related to #3);
  • Map (related to #6).

You can see a demo here (available since 16:00 of 09/10/2017).
You can find additional information in the README.md file.

Screenshot

Here some screenshots of the various components.

Map

Display CSV data in a map
map1

Display CSV data in a map with a detailed table showing on click on a marker
map2

Table

Display CSV data in a table.
table

Continuous Integration

Attached Travis and Codecov to automatize tests execution and code coverage. On top of the README.md file you will also find the related badges or you can see here below:

Build Status
Coverage

Current tested components:

  • CsvTable;
  • TitleSeparator;
  • TitleSeparatorVert.

Way to improve

  • Add more tests;
  • Fix all the warnings (already there);
  • Marker Clustering.

Saluti da Davide e Francesco 馃槃

FranzPichierri and others added some commits Oct 8, 2017

Merge branch 'csv-table' of https://github.com/DavidePastore/daf-data鈥
鈥ortal-public into csv-table

# Conflicts:
#	src/components/CsvDisplayer/CsvDisplayer.js
Merge branch 'csv-table' of https://github.com/DavidePastore/daf-data鈥
鈥ortal-public into csv-table

# Conflicts:
#	src/components/CsvDisplayer/CsvDisplayer.js
#	src/container/Map.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment