search

entdec / signum

Rails engine with replacement for flash messages, Signum allows for messages from background jobs.
MIT License
2 stars 0 forks source link

current-user has the potential to not be defined #23

Closed tdegrunt closed 4 months ago

tdegrunt commented 5 months ago

See also the SignalController:

# signals = Signum.config.current_user.call.signals.where.not(state: "closed") // current_user is nil when called from an api

This is happening because we a) use ActionController::API, though we can just use ApplicationController and b) Signum's application controller doesn't use the base_controller configuration.

rparse commented 5 months ago

PR created - https://github.com/entdec/signum/pull/24

sanderhelsloot commented 4 months ago

Signum messages and notification functionality work as expected.