Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

HsunPei Wang #18

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

HsunPei Wang #18

wants to merge 16 commits into from

Conversation

hsunpei
Copy link

@hsunpei hsunpei commented Oct 27, 2019

Features

  • Display a list of calls fetch from https://aircall-job.herokuapp.com/activities
  • Display the detailed data of a call from https://aircall-job.herokuapp.com/activities/:id
  • Archive a call by calling https://aircall-job.herokuapp.com/activities/:id
  • Reset all calls and show the full list of calls

Screenshots

List of calls

Screen Shot 2019-10-27 at 9 42 54 PM

Detail view

Screen Shot 2019-10-27 at 9 45 35 PM

Archive a call

Kapture 2019-10-27 at 21 49 33

Reset all calls

Kapture 2019-10-27 at 21 51 21

Tooling

  • React
  • Redux-saga
  • styled-components

Todos

  • Add react-router to show different URLs for the detail pages
  • Refine the display and handlings of the data loading state
  • Handle the possible network errors
  • Group the calls by date
  • Add some animations for transitioning between pages or archiving items

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant