This pull request fixes a TODO of renaming the getPost function to getPosts in our Rogue service, and uses it within a getPost function to the ImportRockTheVoteRecord job, adding tests to verify the filters passed to the GET /v3/posts request.
How should this be reviewed?
👀
Any background context you want to provide?
It'll be handy to have this new getPost function within the ImportRockTheVoteRecord job when we eventually support multiple voter registration posts, and will need to inspect the details of each post to determine whether we should create or update a post based on the Started registration column.
What's this PR do?
This pull request fixes a TODO of renaming the
getPost
function togetPosts
in our Rogue service, and uses it within agetPost
function to theImportRockTheVoteRecord
job, adding tests to verify the filters passed to theGET /v3/posts
request.How should this be reviewed?
👀
Any background context you want to provide?
It'll be handy to have this new
getPost
function within theImportRockTheVoteRecord
job when we eventually support multiple voter registration posts, and will need to inspect thedetails
of each post to determine whether we should create or update a post based on theStarted registration
column.Relevant tickets
Prep for Pivotal #171737617.
Checklist