rokwire / illinois-app

Source code repository of "Illinois" App - the official mobile app of the University of Illinois.
https://app.illinois.edu/
Apache License 2.0
23 stars 20 forks source link

[USABILITY] Broken links in Wellness section #3664

Closed joegrohens closed 1 year ago

joegrohens commented 1 year ago

Describe the usability issue

The following links are broken in the WELLNESS section. (Error "Empty Guide Content".)

  1. Wellness Resources > 24 Hour Crisis Resources Should open Campus Guide > Health & Wellness > General Support > 24-Hour Crisis Resources (Content ID: 9214bef9-4cb3-4596-ae1f-aa6e8261d2f0)

  2. Wellness Resources > Counseling Center Should open Campus Guide > Health & Wellness > Mental & Emotional Health > Counseling Center (Content ID: 996915de-2f43-40e1-b24a-40ec7be67c35.1)

  3. Wellness Resources > Covid-19 Link to https://covid19.illinois.edu/ (open in external browser)

joegrohens commented 1 year ago

Note - these are seen broken in prod v4.3.61

TodorBachvarov commented 1 year ago

Fixed though the Admin app for both Dev and Prod, so probably there is no need of new build. But If they still appear broken please test with version 5.0.57

joegrohens commented 1 year ago

@TodorBachvarov @pmarkhennessy

Counseling Center is still broken in v4.3.61, but fixed in v5.0.57.

I don't think that Campus Guide articles are in sync between dev and prod. Counseling Center Content ID differs.

Will Campus Guide content be ported from Dev to Prod before release? I am concerned about what might happen.

Status report after testing.

  1. Wellness Resources > 24 Hour Crisis Resources: fixed in v4.3.61; fixed in v5.0.57
  2. Wellness Resources > Counseling Center: still broken in v4.3.61; fixed in v5.0.57
  3. Wellness Resources > Covid-19: fixed in v4.3.61; menu removed in v5.0
mihail-varbanov commented 1 year ago

Currently we cannot fix the Wellness Resources > Counseling Center link in 4.3.61 because it is hardcoded in the application binary. The only way to resolve it for 4.3.61 is to duplicate the "996915de-2f43-40e1-b24a-40ec7be67c35" guide content entry and make it accessible as "65b51137-bf11-45e8-bdc7-93c160560dcd" ID:

{
  "_id": "65b51137-bf11-45e8-bdc7-93c160560dcd",
   "content_id": "65b51137-bf11-45e8-bdc7-93c160560dcd",
   "content_ref": "7e564d55-6f6e-4097-aeec-76d306cf062d"
}

but this is kind of a patch that I am not sure is worth the noise that it will create. Starting from version 5.0 the wellness resource content resides in our Content BB so we can avoid situations like this.

Yes, we need to syn dev and prod guide content. I am not sure who is responsible for maintaining this.

joegrohens commented 1 year ago

@mihail-varbanov @TodorBachvarov

Since we expect to release v5 soon, I think it’s best to leave this error with the Counseling Center alone.

I’m leaving the issue open to remind me to check in the prod build.

joegrohens commented 1 year ago

Status report for this issue on 5.0.68 prod

Wellness Resources > 24 Hour Crisis Resources: confirmed fixed in v5.0.68 prod Wellness Resources > Counseling Center: confirmed fixed in v5.0.68 prod Wellness Resources > Covid-19: confirmed fixed in v5.0.68 prod|

Closing this issue.