Closed dan-tang-ssd closed 7 months ago
Perhaps we keep it unmerged, as you suggest, and close it. But we keep it here for reference - I'm sure having access to all these exchange rate data in a Laravel package will come in useful again! (It'd be nice to extract this and make it a resource for others too at some point).
Yes, I agree. I will close this PR now.
This PR is submitted to add historic exchange rate data.
Staging env deployment and live env deployment are done in a manual way.
Therefore, it is not necessary to merge this PR for deployment. (Except that we should add a migration file to add a composite index for base_currency_id, target_currency_id, date in exchage_rates table)
This PR contains below changes: