search

adventures-in / chat_app

A group effort to build a chat app and learn as we go.
18 stars 0 forks source link

i144: Add an extension on BuildContext to simplify syntax for accessing DatabaseService #163

Closed nickmeinhold closed 4 years ago

nickmeinhold commented 4 years ago

Fixes #144

This is more of a style choice than anything else, personally I think it improves the clarity and readability of the code but open to opposing opinions, putting the PR up for discussion.

gaslitbytech commented 4 years ago

@nickmeinhold My branch for #168 is touching upon this so if this can be merged sooner rather then later it would be good.

nickmeinhold commented 4 years ago

thanks @tourdownunder, hope it didn't hold you up :-)

whether or not we keep the general policy of leaving PRs for the author to merge, please feel free to merge any of mine if it keeps things moving