Ingress documentation

[gtfierro]

No description provided.

[codecov-commenter]

Codecov Report

Patch and project coverage have no change.

Comparison is base (12ebb4e) 72.75% compared to head (6883bed) 72.75%.

❗ Current head 6883bed differs from pull request most recent head f940be0. Consider uploading reports for the commit f940be0 to get more accurate results

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #229   +/-   ##
========================================
  Coverage    72.75%   72.75%           
========================================
  Files           32       32           
  Lines         2063     2063           
========================================
  Hits          1501     1501           
  Misses         562      562           

Impacted Files	Coverage Δ
buildingmotif/ingresses/base.py	0.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[MatthewSteen]

Looking good! Just a few organizational comments.

Handler vs. Manager? See #14 (comment). I'm not sure what's a better name and know handler is frequently used but I like manager more for some reason. Thoughts? Not sure if there's a best practice or reference for this.

[gtfierro]

Looking good! Just a few organizational comments.

Handler vs. Manager? See #14 (comment). I'm not sure what's a better name and know handler is frequently used but I like manager more for some reason. Thoughts? Not sure if there's a best practice or reference for this.

"Manager" was doing something different there. I like the word "handler" here because it is handling the receipt of information and processing it; it is not mediating or managing access to that information