Spoke: Feature Request: Query-preserving url addressability for MessageReview

Created on 16 Jun 2020  路  2Comments  路  Source: MoveOnOrg/Spoke

Problem
A message review query must be manually selected each time. Reloading clears a query -- and the default query is not even 'cheap' on the backend.

Solution
Message Review is quite powerful, and would be even more so, if queries were addressable. This would enable:

  • Linking from the CampaignStats page directly to a message review for that particular campaign
  • Linking from a texter directly to all their conversations
  • An Admin bookmarking some important queries that they review regularly.

Tech context:

This is the component where filters are chosen and updated:
https://github.com/MoveOnOrg/Spoke/blob/stage-main-71/src/components/IncomingMessageFilter.jsx

The query is run from here:
https://github.com/MoveOnOrg/Spoke/blob/stage-main-71/src/components/IncomingMessageList/index.jsx#L326

We have a few similar url places in the code to copy from:
https://github.com/MoveOnOrg/Spoke/blob/stage-main-71/src/containers/AssignmentTexterContact.jsx#L369
https://github.com/MoveOnOrg/Spoke/blob/stage-main-71/src/components/AssignmentTexter/ContactController.jsx#L41

A-message review C-enhancement S-in progress good first issue help wanted

All 2 comments

I will take a look at this one!

Looks like the stage-main-71 branch was merged, so the links don't work but I think I've found most of the places referenced and gotten a pretty good understanding of things. Definitely seems very doable to me :) @ibrand feel free to officially assign haha

Was this page helpful?
0 / 5 - 0 ratings