Hi Saurav,

I've reviewed your pull request and it looks good to me.

One concern I have is with the UI layout. I think with this many forms it will 
make the statistics page rather long. I wonder if maybe a tab-based layout or 
an accordion style layout might work better?

Another concern is that the date filtering is repeated for each of these 
statistics views. I wonder, why not just have them all utilize the main date 
range?

Are you working from a UX design?

Thanks,

Marcus

> On Aug 3, 2023, at 12:44 PM, saurav kumar jha <imsauravgaurav...@gmail.com> 
> wrote:
> 
> Hi Community,
> 
> I am updating my commits for AIRAVATA-3594 on django-portal in this pull 
> request .
> Any Comments or Recommendations will be very helpful.
> 
> Thanks & Regards,
> Saurav Kumar Jha

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to