Mattermost-server: [Help Wanted] [MM-9602] Move getFlaggedPosts action from webapp to mattermost-redux

Created on 23 Mar 2018  路  2Comments  路  Source: mattermost/mattermost-server

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide, specifically for machine setup and for developer workflow.


Notes: Jira ticket

The webapp has an implementation of the getFlaggedPosts action here: https://github.com/mattermost/mattermost-webapp/blob/master/actions/views/rhs.js#L127

This action belongs in mattermost-redux so it can be shared between multiple apps.

A few things should be done:

  • Create a new action in actions/posts.js of the mattermost-redux repository
  • Make the action follow the same pattern as other mattermost-redux actions
  • Add a unit test for this action in test/actions/posts.js
  • Submit a PR with these updates to mattermost-redux
  • Once merged, update the webapp to use the new mattermost-redux action

Speak to @joram on pre-release (@jwilander on GitHub) for help if needed

Easy Hacktoberfest TecReactJS TecRedux

Most helpful comment

I can take a look.

All 2 comments

I'll take this one!

I can take a look.

Was this page helpful?
0 / 5 - 0 ratings