Commit Graph

10 Commits

Author SHA1 Message Date
Mehak Nasir
c7fd33f99e fix: division scheme is handled on enabling cohorts 2022-01-03 13:44:09 +05:00
AsadAzam
3c6e50bfbc fix: fixed course detail api params (#226)
* fix: fixed course detail api params

* test: fixed tests
2021-12-23 11:24:01 +05:00
Kshitij Sobti
6a2e1ba45f fix: improve styling and behaviour to match mockups [BD-38] [TNL-8730] (#193)
* fix: improve styling and behaviour to match mockups
This fixes the following issues:
TNL-8737: Description field should be larger
TNL-8738: Increase right and left padding in expanded card
TNL-8739: Expand/Collapse icon should have fixed position
TNL-8741: Remove field helper text on field error state
TNL-8742: Use consistent spacing between field and helper text
TNL-8743: Too much spacing above Save button
TNL-8744: remove field error states when re-focusing on field
TNL-8762: Copy changes to team configuration
TNL-8763: Radio button rendering issues
TNL-8764: Use lowercase "M" for "Public managed" and "Private managed" labels
TNL-8766: Match style of labels in collapsed card
2021-10-06 12:58:44 +00:00
Kshitij Sobti
5cc2251af2 apply review feedback 2021-09-17 14:34:40 +05:00
João Cabrita
1a9f2b813a feat: Optionally display message on 403 and disable changing provider after course started [BD-38] [TNL-8142][BB-4253] (#126)
* feat: Prevent changing discussion providers after the course started.
* refactor: confirmation modal and update design according to Figma

Co-authored-by: Awais Ansari <awais.ansari63@gmail.com>
2021-08-11 15:00:28 +05:00
Kshitij Sobti
0e7340bdda feat: Hook up course apps API and add base components for app configuration
Hooks up the course apps API so that the data returned by the server is being used.
Adds the base components and infrastructure to enable adding pages for configuring
each app.
2021-07-01 20:58:05 +05:00
David Joy
9c63ab8044 Refactoring and organization (#41)
* Updating dependencies and removing unneeded ones.

* Fixing broken IntlProvider attribute in ProctoredExamSettings test.

* package-lock.json was out of sync - checking it in.

* Initializing an empty redux store.

* Adding model-store from frontend-app-learning.

This will let us save data from the server in a normalized way in redux, reducing boilerplate in React components.

* Fixing paragon button usage.

(also just organizing the imports while I was there…)

* Using paragon button instead of an anchor tag.

For the “New Page” button in the pages & resources view.

* Add API, reducers, and thunks to add course detail data into redux.

Subsequent PR will use this to store course detail data for use across different pages in the application.

* Prep work to add CourseAuthoringPage component.

Decided the course-detail sub-directory didn’t make much sense, given component structure, and moved it up to src.

These functions will be used in a CourseAuthoringPage component to load course detail data and display the Header and Footer in one common place, wrapping all the existing course authoring pages (proctoring and pages & resources)

It will also replace LmsApiService.js

* Minor style refactorings.

(This commit had originally made some changes to how courseId was passed in to these two components, but I decided to back it out… but the style stuff is worth adding as a fixed nit.)

* Refactor course detail loading and top-level course authoring components

This commit does a few things:

- Factors course detail data loading out of the Header.
- Loads that data in CourseAuthoringPage instead, adding it to redux and then passing it to the Header from there.
- Deletes LmsApiService, which is no longer used.
- Changes the route paths to be more canonical and entity-oriented, i.e., the first part of the route is the course, followed by the specific page about that course to load, rather than the other way around.  This more naturally allows us to use react-router to extract the common course detail loading code that only depends on the courseId.

* Refactoring routes code a bit to pass courseId into components

Didn’t like how CourseAuthoringPage, LegacyProctoringRoute, and CourseAuthoringRoutes all reached into the parent route to find the courseId, so passed it in instead.

* Updating README with more detail on routes in the MFE.
2021-01-07 13:16:35 -05:00
Michael Roytman
309aa93607 [WIP] New Header Work (#33)
* update Studio header to include dropdown menus and internationalization

* Added header improvements (#34)

Added header improvements

Squashing commits for header improvements

added header improvements

fixed typo

moved api call to separate file

added course lockup to mobile header, removed snapshot tests

fixed css for mobile header

simplified css styling

updated testing

updated css styling

updated css

simplified course lockup

removed React fragments from lockup

fixed mobile header styling

Co-authored-by: alangsto <46360176+alangsto@users.noreply.github.com>
2020-08-13 11:43:01 -04:00
Simon Chen
9352ab3178 MST-334 Implement the settings post back functionality (#16)
* MST-334 Implement the settings post back functionality

* fix

* take in feedback

* feedback
2020-07-24 16:46:53 -04:00
Michael Roytman
500eba661e add Studio base URL to configuration and add Proctored Exam Settings component (#4) 2020-07-14 14:24:30 -04:00