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.

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

Error message in MarketDataFxRateProvider doesn't include data source name #1225

Closed
cjkent opened this issue Jul 15, 2016 · 0 comments
Closed

Error message in MarketDataFxRateProvider doesn't include data source name #1225

cjkent opened this issue Jul 15, 2016 · 0 comments
Assignees
Milestone

Comments

@cjkent
Copy link
Member

@cjkent cjkent commented Jul 15, 2016

The exception thrown when the FX rate isn't available "No market data available..." includes the currencies but not the ObservableSource used to look up the data. This makes debugging hard if the data is missing.

@jodastephen jodastephen added this to the v1.1 milestone Jul 22, 2016
jodastephen added a commit that referenced this issue Jul 22, 2016
Fixes #1225
@jodastephen jodastephen self-assigned this Jul 22, 2016
@cjkent cjkent closed this in #1238 Jul 22, 2016
cjkent added a commit that referenced this issue Jul 22, 2016
@cjkent cjkent removed the Status:InReview label Jul 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.