Clarify the HostStatus service in name and purpose

Created on 14 April 2023, almost 2 years ago

Problem/Motivation

The class name and comments for \Drupal\disable_login_by_domain\HostStatus are confusing. It is not clear what exactly this class does.

Proposed resolution

- Add an interface for this class, including clear usage and purposes.
- Make this class implement this interface.
- Consider renaming the class from HostStatus to something like DisableDomains or something similar.

Remaining tasks

- Do the above

API changes

- Class name may change

✨ Feature request
Status

Needs work

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States krisahil

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024