# Review Request: PR #2889 - Restrict Patient Image Upload **URL:** \ **Category:** Development\ **Created:** [February 24, 2026, 5:42pm UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009 "2026-02-24T17:42:14Z")\ **Posts on this page:** 6\ **Page:** 1
**Author:** ![Taniya](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/taniya/32/1186_2.png) [@Taniya](https://talk.openelis-global.org/u/Taniya)\ **Post date:** [February 24, 2026, 5:42pm UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/1 "2026-02-24T17:42:14Z")
Hi everyone, I am looking to contribute to OpenELIS Global and recently raised a PR #2889 to restrict patient image upload. The PR adds a configuration property that allows admins to fully disable patient image uploads, with updates to both the frontend and backend to respect this setting. I have also attached screenshots showing the changes. It has been a couple of days but I haven’t heard back on a review yet. Could someone from the team please take a look when they get a chance? If there is anything that needs to be changed or improved, please let me know and I will get it fixed as soon as possible. Thank you! ---
**Author:** ![Moses\_Mutesasira](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/moses_mutesasira/32/72_2.png) [@Moses\_Mutesasira](https://talk.openelis-global.org/u/Moses_Mutesasira)\ **Post date:** [February 25, 2026, 9:54pm UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/2 "2026-02-25T21:54:12Z")
Thanks @Taniya . can you add a screen recording of your new changes ?? ---
**Author:** ![Taniya](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/taniya/32/1186_2.png) [@Taniya](https://talk.openelis-global.org/u/Taniya)\ **Post date:** [February 28, 2026, 6:06am UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/3 "2026-02-28T06:06:54Z")
I have attached a screen recording. Please check it here [Feat(2889) - Add configuration to disable patient image file upload by taniy8 · Pull Request #2968 · DIGI-UW/OpenELIS-Global-2 · GitHub](https://github.com/DIGI-UW/OpenELIS-Global-2/pull/2968) ---
**Author:** ![priyanshu56](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/priyanshu56/32/1148_2.png) [@priyanshu56](https://talk.openelis-global.org/u/priyanshu56)\ **Post date:** [March 7, 2026, 5:07pm UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/4 "2026-03-07T17:07:29Z")
I noticed that this PR includes a very large number of changes across multiple files. It seems that along with the feature implementation, many formatting updates were also introduced. Because of this, it becomes difficult to clearly review the actual changes related to the solution for this issue. It might be helpful to separate formatting changes from the feature implementation so reviewers can focus on the functional updates more easily. @Taniya sir I review this pr it have formetting changes and some import updation This makes the diff quite large and harder to review. @Moses_Mutesasira ---
**Author:** ![Taniya](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/taniya/32/1186_2.png) [@Taniya](https://talk.openelis-global.org/u/Taniya)\ **Post date:** [March 8, 2026, 9:05am UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/5 "2026-03-08T09:05:28Z")
Thanks for the feedback. I’ve addressed this. The PR has been cleaned up and now only shows the 8 files directly related to the feature implementation. The formatting noise has been removed and the diff is much easier to review now. Please take another look at PR #2968! @priyanshu56 ---
**Author:** ![priyanshu56](https://yyz2.discourse-cdn.com/flex030/user_avatar/talk.openelis-global.org/priyanshu56/32/1148_2.png) [@priyanshu56](https://talk.openelis-global.org/u/priyanshu56)\ **Post date:** [March 8, 2026, 5:53pm UTC](https://talk.openelis-global.org/t/review-request-pr-2889-restrict-patient-image-upload/2009/6 "2026-03-08T17:53:28Z")
I reviewed this. it looks good