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

Update README for latest version of data (9-11) #719

Closed
jonespm opened this issue Sep 25, 2019 · 5 comments
Closed

Update README for latest version of data (9-11) #719

jonespm opened this issue Sep 25, 2019 · 5 comments
Labels
🪳 bug Something isn't working 📖 documentation Involves updates to the README, CONTRIBUTING, or other supporting text files

Comments

@jonespm
Copy link
Member

jonespm commented Sep 25, 2019

Thank you for contributing to this project!

  • Make sure to search the issues for duplicates first!

Expected behavior (A clear and concise description of what you expected to happen) :

It was noted on the call that myla_test_data_2019_09_11 file is not correct with the current README. We need to update the README for the data in this file.

I'd like to figure out a process for the depersonalizer that will keep some id's static so we don't have to make so many documentation updates.

@jonespm jonespm added the 🪳 bug Something isn't working label Sep 25, 2019
@project-bot project-bot bot added this to To do in MyLA-Default-Project Sep 25, 2019
@ssciolla ssciolla self-assigned this Dec 9, 2019
@ssciolla
Copy link
Contributor

ssciolla commented Dec 9, 2019

We had an EECS student try to update the README to better reflect the changing course id values. In thinking about those changes (which I had them remove), I had the following idea:

I feel like with the changes introduced in PR #761, we probably shouldn't be listing any hardcoded course values and just instruct people to use the admin interface. So instead, under Logging in as an administrator, we'd have the following:

  1. As you are now logged in as root, there are no courses listed. To view the Django admin interface, navigate to http://localhost:5001/admin
  2. Click on "Courses" under "Dashboard" to view a list of sample courses.
  3. Click on the link in the "Course Link" column for the sample course you wish to view.

@jonespm, thoughts?

@ssciolla
Copy link
Contributor

ssciolla commented Dec 9, 2019

Wups, I guess some of this has already been addressed by PR #811. I still think relying on the Django admin might be a good idea.

@ssciolla ssciolla added this to To do in MyLA-Winter-2020.01.02 via automation Feb 12, 2020
@ssciolla ssciolla removed this from To do in MyLA-Default-Project Feb 12, 2020
@ssciolla ssciolla removed this from To do in MyLA-Winter-2020.01.02 Feb 12, 2020
@ssciolla
Copy link
Contributor

ssciolla commented Mar 9, 2020

@lsloan, we talked about updating 5. under "Logging in as administrator" so that it doesn't reference specific course ids but just gives instructions on finding and viewing the course via the Admin. This is to eliminate the need to repeatedly update time-related information.

@ssciolla ssciolla assigned lsloan and unassigned ssciolla Mar 9, 2020
@jennlove-um jennlove-um added this to To do in MyLA-Default-Project via automation Mar 9, 2020
@jennlove-um jennlove-um removed this from To do in MyLA-Winter-2020.01.03 Mar 9, 2020
@jennlove-um jennlove-um added the 📖 documentation Involves updates to the README, CONTRIBUTING, or other supporting text files label Sep 24, 2020
@pushyamig pushyamig removed their assignment Oct 2, 2020
@jennlove-um
Copy link
Contributor

@ssciolla Is this addressed in #661 ? Can we close this issue?

@ssciolla
Copy link
Contributor

Looks like this has been handled by my PR or another earlier on. The README.md doesn't reference a specific file, just the Google Drive directory. I'll close it.

MyLA-Default-Project automation moved this from To do to Review/QA Oct 11, 2022
@ssciolla ssciolla removed this from Review/QA in MyLA-Default-Project Oct 11, 2022
@ssciolla ssciolla added this to To do in MyLA-2022.02.01 via automation Oct 11, 2022
@ssciolla ssciolla moved this from To do to Review/QA in MyLA-2022.02.01 Oct 11, 2022
@jennlove-um jennlove-um moved this from Review/QA to Done in MyLA-2022.02.01 Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪳 bug Something isn't working 📖 documentation Involves updates to the README, CONTRIBUTING, or other supporting text files
Projects
No open projects
Development

No branches or pull requests

5 participants