codeforpdx / PASS

PASS project - with HMIS module integration
MIT License
25 stars 24 forks source link

[Refactor] - Refactor useNotification/useContext with NotificationContext #508

Closed leekahung closed 8 months ago

leekahung commented 8 months ago

Describe the Current Behavior/Feature:

Currently, the useNotification hook is simply a recalling of useContext with the NotificationContext and doesn't do anything more. It'll be simpler to just use useContext in that case. However, we could also attempt to migrate fully what we already have inside the useContext hook with NotificationContext over to useNotification. That way we can clear up some redundant coding.

Rationale:

Cleaning up codebase.

Proposed Implementation (if applicable):

For this issue, we either:

leekahung commented 8 months ago

Actually, might not need this. See https://github.com/codeforpdx/PASS/pull/497#discussion_r1390287244.