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

UI #4

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

UI #4

wants to merge 22 commits into from

Conversation

MarinEstrada
Copy link

No description provided.

Readme now has instructions to run locallly using docker,
gitignore now ignores .swp files
In src/actions/shared.js
    fix getInitialData typo (missing 'i')
In src/reducers/index.js
    remove 'function' from 'export default function'
    when functions called as default, do not need to surround with '{}'
In src/reducers/users.js
In src/reducers/tweets.js
    actions -> action on ln8
also fix typo it 'initial'in src/actions/shared.js
Initial setup of app and of initial data acquisition
render a list  of tweet id's to render when page is fully loaded
later these tweet ids will be exchanged for tweet containers
Added unnder src/components/NewTweet.js
Shows specified tweet, all its replies, and ability to reply to tweet.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant