Skip to content
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.

Sign up for GitHub

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

Add "driver" functionality #84

Merged
merged 16 commits into from
Jan 29, 2023
Merged

Add "driver" functionality #84

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
94dcdde
Added a new interface for the exchange rate drivers to implement.
ash-jc-allen Dec 13, 2022
f3dd736
Moved the exchangeratesapi.io driver to a new folder.
ash-jc-allen Dec 13, 2022
8e167b5
Added the functionality to define a driver in the config.
ash-jc-allen Dec 13, 2022
a778a71
Renamed the folder.
ash-jc-allen Dec 13, 2022
623062d
Updated the class to be a manager rather than being coupled to the API.
ash-jc-allen Dec 13, 2022
5fc1e82
Make the base URL a constant rather than a property.
ash-jc-allen Dec 13, 2022
a52721c
Use the `Http` facade rather than Guzzle.
ash-jc-allen Dec 13, 2022
8b4717b
Removed unneeded constructor.
ash-jc-allen Jan 28, 2023
d41f21e
Lifted shared cache code into trait.
ash-jc-allen Jan 28, 2023
d3398fd
Moved driver-specific tests into new folder.
ash-jc-allen Jan 28, 2023
fd4ce94
Added tests for the manager class.
ash-jc-allen Jan 28, 2023
07ca178
Merge branch 'master' into feature/newer-api
ash-jc-allen Jan 28, 2023
2a3926c
Updated tests to use the new driver class.
ash-jc-allen Jan 28, 2023
85df0cb
Move "throw" to the response than the PendingRequest.
ash-jc-allen Jan 29, 2023
4068e3d
Apply fixes from StyleCI
StyleCIBot Jan 29, 2023
cd0e694
Merge pull request #85 from ash-jc-allen/analysis-VBb0e6
ash-jc-allen Jan 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions config/laravel-exchange-rates.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,12 @@
| API Base URL
|--------------------------------------------------------------------------
|
| Define the URL for the API. Please note that the exchangeratesapi.io
| API only allow HTTPS requests if you are using a paid account. So,
| if you are using a free account, please make sure that your URL
| begins with 'http://'.
| Define which API service should be used to retrieve the exchange rates.
|
| Supported: "exchangeratesapi-legacy", "exchangeratesapi"
|
*/
'api_url' => env('EXCHANGE_RATES_API_URL', 'http://api.exchangeratesapi.io/v1/'),
'driver' => 'exchangeratesapi-legacy',

/*
|--------------------------------------------------------------------------
Expand Down
Loading