Skip to content

Return 'destination' only TDS Reports#1

Open
paulespinosa wants to merge 1 commit into
SmarterApp:masterfrom
paulespinosa:irp-tds-report-extraction
Open

Return 'destination' only TDS Reports#1
paulespinosa wants to merge 1 commit into
SmarterApp:masterfrom
paulespinosa:irp-tds-report-extraction

Conversation

@paulespinosa

Copy link
Copy Markdown

IRP uses TIS to analyze test scores. This change will cause TIS to
return the File ID of the file it is submitting to be scored and return
only a TDS Report if it is marked with the 'destination' location in the
XMLRepository table.

IRP uses TIS to analyze test scores. This change will cause TIS to
return the File ID of the file it is submitting to be scored and return
only a TDS Report if it is marked with the 'destination' location in the
XMLRepository table.
@jtreuting

Copy link
Copy Markdown
Contributor

Thanks Paul. We will review and let you know if we have any questions.

using (HttpClient client = new HttpClient())
{
response = client.GetAsync(String.Format("{0}?access_token={1}", OpenAMUrl, authKey)).Result;
response = client.GetAsync(String.Format("{0}?access_token={1}&realm=/sbac", OpenAMUrl, authKey)).Result;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was it not working for you without the realm? I'm asking because we have stood TIS up and used it and we did not need to change this.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct, it didn't work without the realm. Should I take it out of this pull request?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not yet. I'm going to double check before and after for that since it does seem like it should be there. I'm just surprised that we didn't see an issue with it before. So I'm confused :) . But leave at this point. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants