-
-
Notifications
You must be signed in to change notification settings - Fork 165
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
Courses with names that end with "modelCourse" not appearing on develop. #2384
Comments
I just tested with the develop branch. I created the course Then I tested with the course |
I changed the title to describe what is actually going on. Line 117 of |
Now that I can list those courses after changing that one line, I renamed them to end in |
Yeah, that is a mistake. That should be |
Are we going to want the model course to always have the exact name of |
There is no need for such a thing. Those courses can be hidden if you do not want them to appear on the WeBWorK home page, and they absolutely should appear in other lists in the admin course. So I say no to such an option. |
I see, so that's why it was using Perhaps we should evolve the model course to become an actual course? There could be a course archive file that comes with the distribution, and it could even have the student orientation pre-loaded. |
I don't think that having the model course be an actual course is such a good idea. I don't like the idea of including a course archive in the GitHub repository for one thing. Binary files are always a mess in a git repository. Every time that we change the model course archive, the entire binary file has to be added to the repository, and binary files don't diff so nicely as text files. That results in the repository size growing much faster than for changes to text files. I think that a better idea is to discontinue putting the model course in the courses directory, and instead use it directly from the webwork2 directory. Now that any course can be copied, there really is no reason for the model course to be customizable. A webwork server administrator can create their own actual course (or several of them even) to use for their model course. For someone that is starting from scratch with webwork, the model course is there for them to get started with. Then in future semesters they can start using previous courses as models. There really is no need for the model course to be an actual course. Even if it were a model course the student orientation would not be "pre-loaded" unless the administrator chooses to copy assignments/sets. I don't like the idea of forcing that set to be pre-loaded either. Then if someone doesn't want to use it, they have to delete it. I think there are many like me that find much of the student orientation set not particularly useful or relevant for the way that their courses work. The way it is now is fine. If an instructor wants to use the set it is there for them to import. @somiaj: Sorry for being a bit abrupt with my reply to your idea of a configuration option with a list of courses to not show in the admin course. I just think that having actual courses (with a directory and all database tables) hidden is not a good idea. It makes it difficult to manage those courses, and those courses will still need to be upgraded when webwork is upgraded in addition to updates to course content and such. In addition we have so many things in localOverrides.conf that have been added over the years that are similar to this, and I think in most cases these things are only used by very few. I think that if the admin course had a user interface for editing site settings like this, rather than a system administrator needing to edit a configuration file, then these things would get used more. |
Recently I moved our production server from 2.18 to develop. Prior to the move, we had model courses for classes with
0-
at the start of the course name. For example0-mth060-modelCourse
. This was just a hack to make them appear at the top of the list.Since the upgrade, these courses do not appear in the admin course. They don't appear in Course Listings, and they don't appear in the dropdown list of courses to copy from. But their folders are still there in the
courses/
folder, and their tables are still there in the database. So something is preventing the admin course from "seeing" them, and I guess it has to do with the "0-" at the start of the course name.The text was updated successfully, but these errors were encountered: