-
Notifications
You must be signed in to change notification settings - Fork 558
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
Staging Release #10693
Staging Release #10693
Conversation
…encounter-overview-design
…encounter-overview-design
…encounter-overview-design
…encounter-overview-design
…encounter-overview-design
…encounter-overview-design
…encounter-overview-design
…work/care_fe into encounter-overview-design
…work/care_fe into treatment-summary-ui
…work/care_fe into treatment-summary-ui
…treatment-summary-ui
…istent spacing between options within the popover content (#10636)
…to v5 (#10195) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: rithviknishad <[email protected]>
…ete-and-resource-form-facility-pre-fill Fix autocomplete unable to pre-fill in certain usages; Adds utility fn. to merge autocomplete options; Fixes facility select in resource form edge case
Fixed the cypress failure in platform
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Discharge Summary changes * Minor changes in Encounter overview | Location, Organization * Add feedback for refresh * Mobile fix * Mobile fix again --------- Co-authored-by: rithviknishad <[email protected]>
Co-authored-by: Gigin George <[email protected]>
<> | ||
<div className="flex flex-1 items-center justify-center rounded-lg"> | ||
<img | ||
src={preview || imageUrl} |
Check warning
Code scanning / CodeQL
DOM text reinterpreted as HTML Medium
DOM text
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix AI 2 days ago
To fix the problem, we need to ensure that the preview
and imageUrl
variables are safe to use in the src
attribute of the img
tag. This can be achieved by validating the file type and ensuring that the URLs are properly sanitized before being used.
- Validate the file type to ensure it is an image before setting the
selectedFile
. - Use a safe method to set the
src
attribute of theimg
tag, ensuring that the URL is not interpreted as HTML.
-
Copy modified lines R119-R124 -
Copy modified line R204
@@ -118,3 +118,8 @@ | ||
} | ||
setSelectedFile(e.target.files[0]); | ||
const file = e.target.files[0]; | ||
if (isImageFile(file)) { | ||
setSelectedFile(file); | ||
} else { | ||
toast.warning(t("please_upload_an_image_file")); | ||
} | ||
}; | ||
@@ -198,3 +203,3 @@ | ||
<img | ||
src={preview || imageUrl} | ||
src={preview ? URL.createObjectURL(selectedFile) : imageUrl} | ||
alt="cover-photo" |
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for care-ohc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@gigincg Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
@ohcnetwork/care-fe-code-reviewers
Merge Checklist