-
Notifications
You must be signed in to change notification settings - Fork 42
TICKET:: HCMPRE-2294,HCMPRE-2256,HCMPRE-2257,HCMPRE-2387 #2298
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
base: console
Are you sure you want to change the base?
Conversation
…reen then move to other screen
Caution Review failedFailed to post review comments. 📝 WalkthroughWalkthroughThe changes integrate HRMS components into the Digit UI application. New import and initialization calls for HRMS have been added in multiple entry point files. Additionally, stylesheet links and SCSS files have been updated or added for HRMS-specific styling. Several UI customization files, proxy routes, and configurations have been restructured. A comprehensive set of new HRMS module components, hooks, services, utilities, constants, and pages have been introduced, supporting operations such as employee creation, detail display, campaign assignment, and search functionalities. Changes
Sequence Diagram(s)sequenceDiagram
participant UI as DigitUI
participant HRMS as HRMS Components
participant Payment as Payment Components
UI->>HRMS: initDigitUI() calls initHRMSComponents()
HRMS-->>UI: HRMS components initialized
UI->>Payment: initDigitUI() calls initPaymentComponents()
Payment-->>UI: Payment components initialized
Suggested reviewers
Poem
✨ Finishing Touches
🪧 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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pitabash-eGov Resolve the code comments
health/micro-ui/web/micro-ui-internals/example/public/index.html
Outdated
Show resolved
Hide resolved
return projectName; | ||
} | ||
|
||
export const getHighLevelProjects = (projectDetails) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pitabash-eGov Please check this if this util is necessary, and if its necessary the Enums are hardcoded, Please refer from boundary Order from boundary search API
...h/micro-ui/web/micro-ui-internals/packages/modules/health-hrms/src/components/Utils/index.js
Outdated
Show resolved
Hide resolved
...h/micro-ui/web/micro-ui-internals/packages/modules/health-hrms/src/components/Utils/index.js
Outdated
Show resolved
Hide resolved
} | ||
|
||
useEffect(() => { | ||
switch (action) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pitabash-eGov Add the actions to MDMS, Create a new master and refer from MDMS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: need to change
...ui/web/micro-ui-internals/packages/modules/health-hrms/src/pages/employee/employeeDetails.js
Show resolved
Hide resolved
|
||
// Fetch project details associated with the staff | ||
const response = await Digit.CustomService.getResponse({ | ||
url: Urls.hcm.searchProject, // API endpoint for project search |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pitabash-eGov useCustomAPIhook can be used, here instead of Digit.CustomService.getRespons
health/micro-ui/web/micro-ui-internals/packages/modules/health-hrms/src/services/urls.js
Show resolved
Hide resolved
UserSearch: "/user/_search", | ||
UserLogout: "/user/_logout", | ||
|
||
Shortener: "/egov-url-shortening/shortener", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pitabash-eGov Add url shortner context path as optional
health/micro-ui/web/micro-ui-internals/packages/modules/health-hrms/src/services/urls.js
Outdated
Show resolved
Hide resolved
…to support the unified-qa in setProxy
case "Landline": | ||
return /^[0-9]{11}$/i; | ||
case "PropertyID": | ||
return /^[a-zA-z0-9\s\\/\-]$/i; |
Check warning
Code scanning / CodeQL
Overly permissive regular expression range Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 13 days ago
To fix the problem, we need to correct the overly permissive range A-z
to accurately match only the intended characters. The correct range should be A-Za-z
to include all uppercase and lowercase letters without including unintended special characters.
- Update the regular expression on line 70 to use
A-Za-z
instead ofA-z
. - Ensure that the functionality remains the same by only changing the character range in the regular expression.
-
Copy modified line R70
@@ -69,3 +69,3 @@ | ||
case "PropertyID": | ||
return /^[a-zA-z0-9\s\\/\-]$/i; | ||
return /^[a-zA-Za-z0-9\s\\/\-]$/i; | ||
case "ElectricityConnNo": |
case "IFSC": | ||
return /^[A-Z]{4}0[A-Z0-9]{6}$/; | ||
case "ApplicationNo": | ||
return /^[a-zA-z0-9\s\\/\-]$/i; |
Check warning
Code scanning / CodeQL
Overly permissive regular expression range Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 13 days ago
To fix the problem, we need to correct the regular expression by replacing the overly permissive range A-z
with the correct range A-Z
. This will ensure that only uppercase letters are matched, along with the other intended characters.
- Locate the regular expression in the file
health/micro-ui/web/micro-ui-internals/packages/modules/health-hrms/src/utils/utlis.js
. - Replace the range
A-z
withA-Z
in the regular expressions on lines 70 and 98. - No additional methods, imports, or definitions are needed to implement this change.
-
Copy modified line R70 -
Copy modified line R98
@@ -69,3 +69,3 @@ | ||
case "PropertyID": | ||
return /^[a-zA-z0-9\s\\/\-]$/i; | ||
return /^[a-zA-Z0-9\s\\/\-]$/i; | ||
case "ElectricityConnNo": | ||
@@ -97,3 +97,3 @@ | ||
case "ApplicationNo": | ||
return /^[a-zA-z0-9\s\\/\-]$/i; | ||
return /^[a-zA-Z0-9\s\\/\-]$/i; | ||
case "Password": |
HRMS modules changes:
Summary by CodeRabbit
New Features
Style Enhancements