602 feature request show professor email

[Troy-bailan-King]

Able to show professor name and email on Coursepage and students can add/drop course on Coursepage. If user want to see couse on Schedule page, need to refresh this page.

[Troy-bailan-King]

Tried to review this pull request however I got this error when I opened the website.

Fix the problem

[zhengw5]

I think it is a pretty useful feature to add because many students email their professors during registration due to conflicts or issues that come up. Choosing the section in this page is also pretty useful but I agree that the schedule should be updated without needing to refresh the webpage.

[zhangn7]

The feature works in that it will show the professor's name next to each section they teach. However, when we put the section on the schedule display, we cannot scroll down to see which professor teaches that section if the class block is too short to fit all the information.

[mlammon11]

This feature is a good addition to the website, however I found a bug where if you search the course, for example CSCI 2500, and add it to you schedule, if you click on the hyperlink to see course details, the professors don't show up.

It is works on my computer, could you give me some graph?

[mlammon11]

Mostly works but for some professors there isn't the correct first and last name, and there's not email. Ex David Goldschmidt teaches CSCI 2500, so If I click on CSCI 2500, "Goldsc" is displayed with no email

[Troy-bailan-King]

Goldsc

Mostly works but for some professors there isn't the correct first and last name, and there's not email. Ex David Goldschmidt teaches CSCI 2500, so If I click on CSCI 2500, "Goldsc" is displayed with no email

Because There are many different name abbreviations that exist in different course databases, I have now added this abbreviation to the professor information and fixed the previous hyperlink problem, try again.

[Troy-bailan-King]

I think it is a pretty useful feature to add because many students email their professors during registration due to conflicts or issues that come up. Choosing the section in this page is also pretty useful but I agree that the schedule should be updated without needing to refresh the webpage.

Now we don't need to refresh

[Troy-bailan-King]

The feature works in that it will show the professor's name next to each section they teach. However, when we put the section on the schedule display, we cannot scroll down to see which professor teaches that section if the class block is too short to fit all the information.

Somebody did it on different PR

[mlammon11]

Yes the hyperlink problem is resolved, but what do you mean by "I have now added this abbreviation to the professor information"? I tried running your branch again and I still see "Goldsc"

[Troy-bailan-King]

Which semester are you testing for

[mlammon11]

Which semester are you testing for

Spring 2022

[Troy-bailan-King]

Which semester are you testing for

Spring 2022

[mlammon11]

Which semester are you testing for

Spring 2022

Oop i guess when I refreshed it I forgot to pull from origin, I can see it now. However, there are other courses facing the same issue such as "ARCHITECTURAL DESIGN STUDIO 2" displays "Sehwail" with no email.

[Troy-bailan-King]

This is because the names of professors in the course database are not stored according to the norm, for example: Goldschmidt is abbreviated as Goldsc and Goldschmidt in different courses. this problem may have to be reorganized in the database, now only one can be found, then one can be added.