Skip to content

Add combinator for optional properties - #7

Open
edsko wants to merge 1 commit into
MedeaMelana:masterfrom
edsko:feature/optProp
Open

edsko wants to merge 1 commit into
MedeaMelana:masterfrom
edsko:feature/optProp

Conversation

@edsko

@edsko edsko commented Feb 28, 2015

Copy link
Copy Markdown

(Note that the implementation does not rely on defaultValue and hence does not rely on any Eq instances.)

@edsko

edsko commented Mar 1, 2015

Copy link
Copy Markdown
Author

Note that if we had the standard StackPrisms for Maybe, then we could just write

optProp :: Json a => Text -> Grammar Obj t (Maybe a :- t)
optProp propName = fromPrism just . prop propName <> fromPrism nothing

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.

1 participant