search

kartoza / feti

Further Education and Training Institute Website
Other
2 stars 15 forks source link

Occupation detail view bug #604

Open myarjunar opened 6 years ago

myarjunar commented 6 years ago

Case 1: one occupation view object can only perform click to open detail and then click to close detail once (without clicking other occupation view or refresh page). It only works if we click on the details text. case_1

Case 2: clicking one occupation view object more than one keep showing loading image, which means it performs a request to retrieve the content each time the object is clicked case_2

Case 3: when one occupation view object is opened, clicking other occupation view object doesn't open related occupation, instead, it closes the previous occupation view. case_3

Case 4: see ticket #610

anitanh commented 6 years ago

hi @myarjunar , the workflow of the occupation view is like this:

  1. User search their desired occupations
  2. When they click on the occupation, they will see the location of the providers on the map (this based on the courses related to the occupation). If you don't see the providers on the map, that means the occupation does not have courses registered for it on the database.
  3. If user wants to see the details of the occupation, the can click on details

peek 2018-07-30 10-58

myarjunar commented 6 years ago

@ann26 thanks for telling me that, I didn't really know about the workflow before, my bad.

I tried the workflow and found something:

  1. It seems like it only shows the location of the providers if only the courses were found in the first pathway. If pathway number 1 didn't have any courses but pathway number 2 (for example) did (Refrigeration Mechanic), it will not show the location, it will show the details instead. Is that the correct behavior?

  2. IMHO, I think it's better if we show a warning like Courses not found when we click on occupation with no courses registered.

anitanh commented 6 years ago

hi @myarjunar no problem, also thanks for your finding.