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

[DEA] Multiselect media support #1133

Closed
Tracked by #1507
mahalakshme opened this issue Feb 14, 2024 · 17 comments
Closed
Tracked by #1507

[DEA] Multiselect media support #1133

mahalakshme opened this issue Feb 14, 2024 · 17 comments
Assignees

Comments

@mahalakshme
Copy link
Contributor

mahalakshme commented Feb 14, 2024

Current status:

Acceptance criteria:

The below should work for Image concept, for video, move it to separate card, if it is difficult(retaining the current functionality). Just added all in the same card, thinking if it is just one day additional effort, then I suggest we can complete it.

  • When the media concept is single select, it should work in the way currently it is working.
  • when the media concept is multiselect, on clicking upload image/video again, additional files should get added.
    --- when editing a form with a multiselect media concept,
    --- I should be able to add an additional image to the existing list of media
    --- I should be able to remove one image from the existing list of media
    --- I should be able to do both the above together
    --- When saving the form, each change mentioned above is handled correctly
    --- If I do not save the form, then changes do not persist
    --- when creating a form with a multiselect media concept, all the above criteria is fulfilled
  • Add the ability to be able to do multiple selection on single click of 'Upload image/video'
  • For viewing the uploaded files, refer the mockups below: The same mockups are applicable for Summary and Recommendations page as well as for viewing observations in DEA. Kindly also refer the design related AC in the violet sticky in the mockup.
    media_viewing.pdf

Tech reasons:

  • Continuing to support carousel for viewing media, is not needed, hence is removed. Also the library currently used for carousel is archived.

Out of scope:

Test cases:

  • SingleSelect and MultiSelect
  • subject registration, program enrolment, program encounter and encounter
  • Create and Edit(addition, deleting) flows
  • Image and Video concepts
  • Upload via mobile and view in DEA
  • Upload via DEA and view in mobile

Old: ignore

----- When multiple files uploaded from mobile, they are not visible in DEA. For them, Additional images uploaded(via DEA/mobile) should be visible, on clicking the right arrow in the carousel that appears when clicking 'View Media'.
Image
----- In the Profile tab, in View mode increase the height of carousel to triple that of current one. On clicking right arrow again here as well, users should be able to see consecutive images. And on clicking cross mark in top right currently unable to close the carousel, though it closes in Edit mode. This need to be fixed.
Screenshot 2024-03-21 at 12 54 59 PM
----- Remove the 'Open in New tab' option.
----- Carousel need to be applicable only for the media in that concept when clicked on View media. Currently it shows media across all concepts for that subject.
https://github.com/avniproject/avni-webapp/assets/10369963/7cdc3b85-f99b-471f-ae55-4d959ae4d0f4

  • For video concept,
    ----- downloads in carousel when uploaded via DEA, works fine when uploaded via mobile app. This need to be fixed.
@mahalakshme mahalakshme converted this from a draft issue Feb 14, 2024
@vinayvenu vinayvenu added the good first issue Good for newcomers label Feb 29, 2024
@mahalakshme mahalakshme added good first issue Good for newcomers and removed good first issue Good for newcomers labels Mar 8, 2024
@mahalakshme mahalakshme removed the good first issue Good for newcomers label Mar 21, 2024
@mahalakshme mahalakshme moved this from In Analysis to Analysis Complete in Avni Product Mar 21, 2024
@vinayvenu
Copy link
Member

@mahalakshme

  • There are a lot of issues mentioned here. This should be broken down into multiple cards each with its own acceptance criteria. This will help us focus on the core issues of each, discuss clearly in the card and test well

Suggesting detailed acceptance criteria to ensure testing does not miss flows. Example:

Where

  • a form is used across subject registration, program enrolment, program encounter and encounter

  • a media concept includes image, video and file concepts that are used in forms

  • As a user,

    • when editing a form with a multiselect media concept,
      • I should be able to add an additional image to the existing list of media
      • I should be able to remove one image from the existing list of media
      • I should be able to do both the above together
      • When saving the form, each change mentioned above is handled correctly
      • If I do not save the form, then changes do not persist
    • when creating a form with a multiselect media concept, all the above criteria is fulfilled
    • when creating a form with a singleselect media concept, existing behaviour is retained

Testing notes

  • Testing required for
    • SingleSelect and MultiSelect
    • subject registration, program enrolment, program encounter and encounter
    • Create and Edit flows
    • Image, Video and file concepts

@mahalakshme
Copy link
Contributor Author

mahalakshme commented Mar 22, 2024

@vinayvenu I can add the test cases that you have mentioned and few more, but I mentioned everything in the same card, because everything is somewhat interrelated.

  • Fixing for image would in itself fix for video - for carousel related issues

May be I ll do the below:

  • Move the file related AC to different card
  • Add the test cases.

@mahalakshme mahalakshme moved this from Analysis Complete to In Analysis Review in Avni Product Mar 22, 2024
@mahalakshme mahalakshme moved this from In Analysis Review to Ready in Avni Product Mar 22, 2024
@mahalakshme mahalakshme changed the title Multiselect media support in DEA [DEA] Multiselect media support Mar 22, 2024
@mahalakshme mahalakshme moved this from Ready to In Analysis in Avni Product Mar 26, 2024
@mahalakshme mahalakshme moved this from In Analysis to Ready in Avni Product Mar 27, 2024
@mahalakshme mahalakshme moved this from Ready to In Analysis in Avni Product Mar 27, 2024
@mahalakshme mahalakshme moved this from In Analysis to Ready in Avni Product Mar 27, 2024
@himeshr himeshr self-assigned this Apr 2, 2024
@himeshr himeshr moved this from Ready to In Progress in Avni Product Apr 2, 2024
himeshr added a commit that referenced this issue Apr 3, 2024
himeshr added a commit that referenced this issue Apr 3, 2024
himeshr added a commit that referenced this issue Apr 3, 2024
…ave as single file upload inline with earlier behaviour
@himeshr
Copy link
Contributor

himeshr commented Apr 3, 2024

@himeshr
Copy link
Contributor

himeshr commented Apr 3, 2024

Create / edit form test cases for Image and Video type formElements

  • Multi-select image allowed
  • Show Thumbnails on selecting single image
  • Show Thumbnails on selecting multiple image
  • Allow preview of images
  • Thumbnail refresh on selecting single videos
  • Thumbnail refresh on selecting multiple videos
  • When the media concept is single select, it should work in the way currently it is working.
  • when the media concept is multiselect, on clicking upload image/video again, additional files should get added.
  • when editing a form with a multiselect media concept,
    --- I should be able to add one or more additional images to the existing list of media
    --- I should be able to remove one or more images from the existing list of media
    --- I should be able to do both the above together
    --- When saving the form, each change mentioned above is handled correctly
    --- If I do not save the form, then changes do not persist
  • Retain checks on media type
  • Retain type and size check on Files(docx, pdf, etc)
  • Behaviour is unchanged for Audio and file formElements
  • Allow preview of video

himeshr added a commit that referenced this issue Apr 4, 2024
himeshr added a commit that referenced this issue Apr 4, 2024
himeshr added a commit that referenced this issue Apr 4, 2024
himeshr added a commit that referenced this issue Apr 4, 2024
@himeshr himeshr moved this from QA Ready to QA Failed in Avni Product May 1, 2024
himeshr added a commit to avniproject/avni-models that referenced this issue May 3, 2024
himeshr added a commit that referenced this issue May 6, 2024
@himeshr himeshr moved this from QA Failed to QA Ready in Avni Product May 6, 2024
@himeshr
Copy link
Contributor

himeshr commented May 6, 2024

Sample screenshots

DEA Summary page:
Screenshot 2024-05-06 at 2 17 38 PM

DEA Form Create / Edit page:
Screenshot 2024-05-06 at 2 17 51 PM

@himeshr himeshr moved this from QA Ready to Code Review Ready in Avni Product May 6, 2024
@himeshr
Copy link
Contributor

himeshr commented May 6, 2024

Only code review commits made since card was moved to "QA Failed" lane.

@petmongrels petmongrels moved this from Code Review Ready to In Code Review in Avni Product May 8, 2024
@petmongrels petmongrels moved this from In Code Review to QA Ready in Avni Product May 17, 2024
@AchalaBelokar
Copy link

AchalaBelokar commented May 27, 2024

  • If Image or video is larger than normal size clear message should be visible but currently this message is showing
error
  • Login with AchalaB@hello and register one teacher which contain multi-select image and video and view and download also it is working fine there
  • try to edit the teacher I am able to see the actual image and video
  • Pending testing in avni-client the images uploaded through DEA

@AchalaBelokar AchalaBelokar moved this from QA Ready to QA Failed in Avni Product May 27, 2024
himeshr added a commit to avniproject/avni-server that referenced this issue May 28, 2024
@himeshr
Copy link
Contributor

himeshr commented May 28, 2024

@AchalaBelokar i have fixed this issue "If Image or video is larger than normal size clear message should be visible"
Please resume testing on webapp and Client

@himeshr himeshr moved this from QA Failed to QA Ready in Avni Product May 28, 2024
himeshr added a commit to avniproject/avni-server that referenced this issue May 28, 2024
@AchalaBelokar AchalaBelokar moved this from QA Ready to In QA in Avni Product May 31, 2024
@AchalaBelokar
Copy link

@himeshr tested on web app with max size upload is showed

@AchalaBelokar AchalaBelokar moved this from In QA to QA Failed in Avni Product Jun 3, 2024
@himeshr himeshr moved this from QA Failed to QA Ready in Avni Product Jun 3, 2024
himeshr added a commit to avniproject/avni-server that referenced this issue Jun 3, 2024
…upload file size is exceeded

(cherry picked from commit 5f2ca33)
himeshr added a commit to avniproject/avni-server that referenced this issue Jun 3, 2024
@himeshr
Copy link
Contributor

himeshr commented Jun 3, 2024

Staging test screenshot after cherry-pick to 9.0 branch:
Screenshot 2024-06-03 at 1 43 09 PM

@AchalaBelokar AchalaBelokar moved this from QA Ready to In QA in Avni Product Jun 3, 2024
@AchalaBelokar AchalaBelokar moved this from In QA to Done in Avni Product Jun 3, 2024
petmongrels pushed a commit that referenced this issue Jun 10, 2024
shraddha761 pushed a commit to shraddha761/avni-server that referenced this issue Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants