Follow Elixir naming conventions by using question mark suffix for boolean
functions. Renamed mail_allowlisted to signup_allowed? and mail_allowlisted_fun
to signup_allowed_fun throughout the codebase.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Define environment variable name as a module attribute in BasicSignupAllowlist
and expose it via signup_allowlist_emails/0 function to eliminate duplication
across lib and test files.
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
Prefactoring, from TDDAYMI to imperative shell. In the next step we
need the implementation from the functional core in the imperative
shell, to pass in the environment variable.
Intermediate steps not committed - addresses_as_list, because long stack
trace in output. turned out 'nil' case needs to be explicit now. I
prefer the small functions though instead of whole implementation in
case statement.
