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

#165273540: Users should be able to follow each other #40

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

nadralia
Copy link
Contributor

@nadralia nadralia commented Jun 20, 2019

What does this PR do?

  • Users should be able to follow and unfollow each other

Description of Task to be completed?

  • Authors should follow each other

  • Authors should unfollow each other

  • Authors should be able to view his/her followers

  • Authors should be able to view his/her followings

How should this be manually tested?

  • Check out git checkout ft-follow-unfollow-165273540

  • npm install && npm run dev

  • log in with your account credentials

  • Append an existing users username to the route to navigate to a users profile
    e.g. https://localhost:8080/profile/{username}

What are the relevant pivotal tracker stories?

Screenshots (if appropriate)

Screenshot 2019-06-21 at 10 13 54

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

@@ -16,6 +16,7 @@ module.exports = {
store: path.resolve(__dirname, 'src/store/'),
utils: path.resolve(__dirname, 'src/utils/'),
constants: path.resolve(__dirname, 'src/constants/'),
routers: path.resolve(__dirname, 'src/routers/'),

Choose a reason for hiding this comment

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

'__dirname' is not defined no-undef

@@ -0,0 +1,11 @@
import axios from 'axios';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,50 @@
import {

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,48 @@
// import action types
import {

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,57 @@
// import action types
import {

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,27 @@
import React from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,88 @@
// react libraries
import React, { Component } from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,32 @@
// react imports
import React from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,182 @@
// react library
import React, { Component } from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,154 @@
// react library
import React, { Component } from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@nadralia nadralia force-pushed the ft-follow-and-unfollow-165273540 branch from 24a11e6 to 5699368 Compare June 20, 2019 05:52
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

@@ -0,0 +1,17 @@
import firebase from 'firebase';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,47 @@
import React from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,25 @@
import React from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@@ -0,0 +1,51 @@
// import react library
import React from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

@nadralia nadralia force-pushed the ft-follow-and-unfollow-165273540 branch from 5699368 to 4ad207e Compare June 20, 2019 05:54
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

The configured stylelint version is not supported.
The configured stylelint version is not supported. 
See a list of supported versions on our docs page.

@nadralia nadralia force-pushed the ft-follow-and-unfollow-165273540 branch from 4726737 to 513f30f Compare June 20, 2019 14:31
Add action creators for follow
Add reducers for follow

[Maintains #165273540]
@nadralia nadralia force-pushed the ft-follow-and-unfollow-165273540 branch from 513f30f to d39b2f8 Compare June 21, 2019 07:10
@@ -0,0 +1,65 @@
// React libraries
import React, { Component } from 'react';

Choose a reason for hiding this comment

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

Parsing error: 'import' and 'export' may appear only with 'sourceType: module'

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