stats4sd / aec_portfolio

A proof of concept for the AEC Consortium Project Management / Assessment System
GNU General Public License v3.0
0 stars 0 forks source link

Fix User Support Issue #266

Closed dan-tang-ssd closed 2 months ago

dan-tang-ssd commented 2 months ago

This PR is submitted to fix #265

This PR contains below changes:

  1. If the "anonymise" option is ticked, column user_feedbacks.user_id should be null. To make sure we are doing what we do.
    • I cannot replicate this issue in local env. column user_feedbacks.user_id is null when "anonymise" option is ticked
    • Maybe we have this bug in early stage, and then we fixed it but it has not been deployed to live env yet
  2. By default, user support request should untick the "anonymise" option.
  3. My Institution > Portfolio > Help text for "Institution Portfolios", we should show the help text all the time. To increase the possibility that user to read it.

Screen shots: image

image

image

what-the-diff[bot] commented 2 months ago

PR Summary

dan-tang-ssd commented 2 months ago

Staging env deployment is completed. Performed testing with positive result.


Screen shots (BEFORE):

image

image


Screen shots (AFTER):

image

image