Skip to content

My opinions: stop following and ignore feature + refactoring#138

Merged
pertrai1 merged 1 commit intowevote:developfrom
lisamburns:my_opinions
Mar 17, 2016
Merged

My opinions: stop following and ignore feature + refactoring#138
pertrai1 merged 1 commit intowevote:developfrom
lisamburns:my_opinions

Conversation

@lisamburns
Copy link
Copy Markdown
Contributor

-Made the My Opinions follow/ignore buttons work to remove items per page (consistent with the More Opinions page).
-Refactored GuideStore that the Opinions/MyOpinions page both use to the new style.
-Resolves setState warnings bug on MyOpinions
-Deleted some now deprecated files.
@DaleMcGrew @nf071590

@DaleMcGrew
Copy link
Copy Markdown
Member

👍 Your changes look good to me, but @nf071590 or @pertrai1 should probably weigh in before it is approved. Great work Lisa!

};

export default GuideActions;
var Dispatcher = require("../dispatcher/Dispatcher");
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

preference would be to import and export. If there is a reason not for doing so, I would say to use let

@pertrai1
Copy link
Copy Markdown
Member

pertrai1 added a commit that referenced this pull request Mar 17, 2016
My opinions: stop following and ignore feature + refactoring
@pertrai1 pertrai1 merged commit a1e337d into wevote:develop Mar 17, 2016
@lisamburns lisamburns deleted the my_opinions branch March 19, 2016 18:35
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.

3 participants