-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Конфигурация пакета #19
Comments
Автор проекта давно уже не отвечает тут, к сожалению. Возможно, вам конечно повезет, но вряд ли. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Предлагаю не навязывать единственный способ конфигурирования (код в конструкторе Handler'а), а давать разработчику возможность определять, каким образом получать ключ API и настройки.
Например, для использования в Laravel по-хорошему домен, логин и ключ нужно брать из .env, а файл с настройками автоматически кешировать. Причём кеш может быть разным в зависимости от окружения (Redis, тупой файловый, в базе и т.п.), поэтому тут тоже ничего лучше не придумывать - просто дать возможность получать сразу массив с конфигом конструктором в качестве параметра.
Могу сделать pull request.
The text was updated successfully, but these errors were encountered: