Skip to content

Add a Field for additional queries with multiple IDs#64

Merged
emhoracek merged 3 commits intomasterfrom
add-list-of-query
Jun 12, 2019
Merged

Add a Field for additional queries with multiple IDs#64
emhoracek merged 3 commits intomasterfrom
add-list-of-query

Conversation

@emhoracek
Copy link
Member

No description provided.

EndpointKey separates params instead of concatting onto endpoint

Make tests pass
@emhoracek emhoracek force-pushed the add-list-of-query branch from f00052a to f343303 Compare June 12, 2019 18:05
@emhoracek emhoracek requested a review from dhartunian June 12, 2019 18:06
spec/Main.hs Outdated
let tpl = toTpl "<wp><wpPostByPermalink><wpDepartment><wpName /></wpDepartment></wpPostByPermalink></wp>"
rendered <- evalStateT (runTemplate tpl [] s mempty) ctxt'
rendered `shouldBe` "Sports"
it "should render stuff 2" $ do
Copy link
Contributor

Choose a reason for hiding this comment

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

any chance we can improve naming here? :)

Copy link
Member Author

Choose a reason for hiding this comment

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

lmao I forgot about that, the previous one should probably be changed from "should render stuff" as well

@emhoracek emhoracek merged commit 62f8522 into master Jun 12, 2019
@emhoracek emhoracek deleted the add-list-of-query branch June 12, 2019 18:39
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.

2 participants