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

Devbranch #3

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Devbranch #3

wants to merge 11 commits into from

Conversation

MikeOxlongfrfr
Copy link
Contributor

Add and edit merge first part, smaller grammar-based changes, some work towards consistency (ALT+X replaced with Alt+X, some screen descriptions re-written to match others,...)

Edit in title of page changed to Add/Edit for each page that had an Add page with an identical name (i.e. Edit changed to Add/Edit in edit-course-offering.md because add-course-offering.md exists..)
other changes:
- "for more details" changed to "for more detail"
- he/she etc replaced with simpler alternatives
- other smaller grammar-based changes
@@ -1,5 +1,5 @@
---
layout: default
title: Add Academic Area
forward: academic-areas
title: add academic area
Copy link
Contributor

Choose a reason for hiding this comment

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

Why was the title changed???

* In this case, curricula from each major are inhereted and can be overridden on a multiple major level (when multiple majors are selected)
* Common courses can be similarly defined on a curriculum with no major
* In Edit Curriculum:
* The major (or more majors) within the selected academic area for which the curriculum is or will be set up 

Copy link
Contributor

Choose a reason for hiding this comment

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

This is not correct! The Add Curriculum comment belongs to the Multiple Majors toggle, which exists on both add and edit pages.

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.

None yet

2 participants