search

Automattic / php-push

Work-in-progress on Push Notifications v2
5 stars 1 forks source link

Load Dotenv in index.php #25

Closed mokagio closed 4 years ago

mokagio commented 4 years ago

I'm not sure whether this is required or there's something missing in my setup.

Even after creating .env locally, php index.php failed saying that the init parameters for APNSConfiguration were booleans instead of strings, that is, the environment variables couldn't be found.

Explicitly loading dotenv as per the documentation solved the problem for me.

codeclimate[bot] commented 4 years ago

Code Climate has analyzed commit 29c1a401 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 80.7% (0.0% change).

View more on Code Climate.

mokagio commented 4 years ago

I merged this because #30 can be rebased on top of it with no conflicts 💃 🕺