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 course name to the output of integration data #230

Open
xploitedd opened this issue May 21, 2021 · 2 comments
Open

Add course name to the output of integration data #230

xploitedd opened this issue May 21, 2021 · 2 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@xploitedd
Copy link
Member

Currently, the Integration data only contains the course acronym, however Core client applications such as i-on Web need the course name, in order to display it to the application users.

An automatic solution for this problem would be to obtain the programme HTML as published by the institution (for instance, the new LEIC programme information) and compare it to the timetable, in order to create a relation between the acronym and the course name. This solution is not foolproof but should reduce the amount of manual work.

Moreover, if the course name cannot be detected by Integration, I believe the course name should be the acronym itself, so that every course has a name, even if it's not the full name.

@grimord grimord added enhancement New feature or request question Further information is requested labels May 21, 2021
@grimord
Copy link
Contributor

grimord commented May 21, 2021

Interesting question. Right now scraping info off ISEL's webpage is on hold as we expect the new website to be released in the near future.

Although, for this particular piece of info, the course name could reasonably be added to our course configuration file, as it's relatively (totally?) static metadata.

@rjcanto , thoughts?

Edit: My original post was a bit off as I was reading "course" but thinking "programme".

@rjcanto
Copy link
Contributor

rjcanto commented May 21, 2021

As @grimord has stated the ability of obtaining the courses names require the implementation of a scraper that can go through the institution's webpage, thus extending significantly the scope of the project for this semester.

It is a good feature to add, but I'm not sure if it can fit the project plan right now. Definitely the priority will be timetables (which for ISEL-ADEETC is almost completed), academic calendar and exam schedule.

I say let's put as enhancement/feature request and see if we can pick up once the primary objectives have been completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants