Skip to content

Commit f7443d6

Browse files
chore: update endpoint and attributes
1 parent fd10550 commit f7443d6

File tree

3 files changed

+9
-25
lines changed

3 files changed

+9
-25
lines changed

src/openResponses/components/DetailAssessmentsList.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const mockResults = [
1515
{
1616
id: '1',
1717
unitName: 'Unit 1',
18-
assessment: 'Assessment 1',
18+
displayName: 'Assessment 1',
1919
totalResponses: 2,
2020
training: 0,
2121
peer: 1,
@@ -44,7 +44,7 @@ describe('DetailAssessmentsList', () => {
4444
});
4545
renderWithIntl(<DetailAssessmentsList />);
4646
expect(screen.getByText(mockResults[0].unitName)).toBeInTheDocument();
47-
expect(screen.getByText(mockResults[0].assessment)).toBeInTheDocument();
47+
expect(screen.getByText(mockResults[0].displayName)).toBeInTheDocument();
4848
expect(screen.getByRole('link', { name: /View and Grade Responses/i })).toBeInTheDocument();
4949
});
5050

src/openResponses/components/DetailAssessmentsList.tsx

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -11,27 +11,10 @@ interface DataTableFetchDataProps {
1111
pageIndex: number,
1212
}
1313

14-
// Example of api response to test on UI
15-
// const mockResults = [
16-
// {
17-
// id: '1',
18-
// unitName: 'Example Unit',
19-
// assessment: 'ORA block number 1',
20-
// totalResponses: 2,
21-
// training: 0,
22-
// peer: 1,
23-
// self: 0,
24-
// waiting: 0,
25-
// staff: 0,
26-
// finalGradeReceived: 1,
27-
// url: 'http://apps.local.openedx.io:8080/instructor/course-v1:DV-edtech+check+2025-05/open_responses'
28-
// }
29-
// ];
30-
3114
const DetailAssessmentsList = () => {
3215
const intl = useIntl();
3316
const { courseId } = useParams();
34-
const [page, setPage] = useState(0);
17+
const [page, setPage] = useState(1);
3518
const { data = { count: 0, results: [] }, isLoading } = useDetailAssessmentsData(courseId ?? '', {
3619
page,
3720
pageSize: DETAILS_PAGE_SIZE
@@ -41,7 +24,7 @@ const DetailAssessmentsList = () => {
4124

4225
const tableColumns = [
4326
{ accessor: 'unitName', Header: intl.formatMessage(messages.unitName) },
44-
{ accessor: 'assessment', Header: intl.formatMessage(messages.assessment) },
27+
{ accessor: 'displayName', Header: intl.formatMessage(messages.assessment) },
4528
{ accessor: 'totalResponses', Header: intl.formatMessage(messages.totalResponses) },
4629
{ accessor: 'training', Header: intl.formatMessage(messages.training) },
4730
{ accessor: 'peer', Header: intl.formatMessage(messages.peer) },
@@ -63,7 +46,7 @@ const DetailAssessmentsList = () => {
6346

6447
return (
6548
<div className="mt-4.5">
66-
<h3>{intl.formatMessage(messages.details)}</h3>
49+
<h3 className="text-primary-700">{intl.formatMessage(messages.details)}</h3>
6750
<DataTable
6851
columns={tableColumns}
6952
data={tableData}

src/openResponses/data/api.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { getAuthenticatedHttpClient, camelCaseObject } from '@openedx/frontend-base';
1+
import { getAuthenticatedHttpClient, camelCaseObject, snakeCaseObject } from '@openedx/frontend-base';
22
import { getApiBaseUrl } from '../../data/api';
33

44
export const getOpenResponsesData = async (courseId: string) => {
@@ -11,7 +11,8 @@ export const getDetailAssessmentsData = async (
1111
courseId: string,
1212
params: Record<string, string | number | boolean> = {},
1313
) => {
14-
const url = `${getApiBaseUrl()}/api/instructor/v2/courses/${courseId}/open-responses/assessments`;
15-
const { data } = await getAuthenticatedHttpClient().get(url, { params });
14+
const url = `${getApiBaseUrl()}/api/instructor/v2/courses/${courseId}/ora`;
15+
const snakeCaseParams = snakeCaseObject(params);
16+
const { data } = await getAuthenticatedHttpClient().get(url, { params: snakeCaseParams });
1617
return camelCaseObject(data);
1718
};

0 commit comments

Comments
 (0)