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

Add Canvas settings URL #3

Conversation

ssciolla
Copy link

@ssciolla ssciolla commented Aug 3, 2021

For consideration to be included in tl-its-umich-edu#164.

@chrisrrowland chrisrrowland merged commit 06dfdfd into chrisrrowland:88_get_sections_fe_be Aug 5, 2021
chrisrrowland added a commit that referenced this pull request Aug 9, 2021
* Wire up get course sections w/ create sections FE

* Wire up adding sections

* Handle error display

* File name alignment fix

* File name alignment fix

* Refactor flow.
Apply context free validation on file upload.
Apply validation requiring awareness of server data on submit.

* Uncoment a thus far unused property

* Update ccm_web/client/src/api.ts

Co-authored-by: Mr. Lance E Sloan «UMich» <lsloan-github.com@umich.edu>

* Non functional tweaking of getPost getPut

* Delete the forbidden comment

* Reorder some imports ~asthetics~

* Remove an unused css style

* rename a variable

* Only change to LoadingExistingSectionNamesFailed if getCanvasSectionDataError is defined

* Prevent multiple submits

* Update ccm_web/client/src/pages/BulkSectionCreate.tsx

Suggested formatting change

Co-authored-by: Sam Sciolla <35741256+ssciolla@users.noreply.github.com>

* Update ccm_web/client/src/pages/BulkSectionCreate.tsx

Suggested formatting change

Co-authored-by: Sam Sciolla <35741256+ssciolla@users.noreply.github.com>

* Update ccm_web/client/src/pages/BulkSectionCreate.tsx

Fix alert text

Co-authored-by: Sam Sciolla <35741256+ssciolla@users.noreply.github.com>

* Escape quotes in text

* Learning to spell basic words

* Get rid of redundant code.  Need to get better at javascript

* Add Canvas settings URL (#3)

* Send Canvas URL in /api/globals; wire up BulkSectionCreate to use it for settings link

* Use Link component; add target _parent

Co-authored-by: Mr. Lance E Sloan «UMich» <lsloan-github.com@umich.edu>
Co-authored-by: Sam Sciolla <35741256+ssciolla@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants