Skip to content

Conversation

@TamilRamGanesan-SF5080
Copy link
Collaborator

Feature description

Need to complete the incomplete feature in Employee Management

Key changes implemented:

  • Need to add the available status in the profile page
  • Need to add the Badge for the leave type
  • Need to add the pagging on the leave report page
  • Need to change the align of the number on the right side of the column
  • Need to add the gap between the header and the value
  • Need to add the background image for the sidebar
  • Added the Responsive for all the pages in the mobile view

Requirement and specification document.

  • Added the available status in the profile page
  • Added the Badge for the leave type
  • Added the paging on the leave report page
  • changed the align of the number on the right side of the column
  • Added the gap between the header and the value
  • Added the background image for the sidebar
  • Added the Responsive for all the pages in the mobile view

API Review task

NA

Output screenshots

image image image image image image

Feature matrix document

Feature matrix document updated against this feature and committed in this common location .

  • Yes
  • NO
  • NA

Provide the details about the areas or combinations which have been tested against this code changes.

  • Tested against feature matrix.

Feature acceptance criteria (Test-case document)

Draft the test cases in excel and attach it in the MR itself. The automation must be covered based on this positive and worst-case test-cases.

Automation details

Mark 'Is Automated' field as (Yes, Manual, Not Applicable) in corresponding JIRA task once the feature is automated.

  • BUnit, share corresponding MR.
  • E2E or Manual Automation using tester - Make sure all items are automated with priority before release which can be tracked in automation dashboard.

Test bed sample location

Provide the test bed sample location where the code reviewers can review the new feature’s behaviors.

Feature completion checklist

UX changes got approval if UI is added or modified?

  • Yes
  • NO
  • NA

Drafted UG for this feature?

  • Yes
  • NO
  • NA

Content team reviewed the UI content changes.

  • Yes
  • NO
  • NA

Is this the common feature which needs to be addressed in the same component or on other components in our platform?

  • Yes
  • NO

Is there any existing behavior change due to this code change?

  • Yes
  • NO

Blazor Checklist

Confirm whether the ensured feature is in both Blazor Server and WASM.

  • Yes
  • NO
  • NA

Do the code changes cause any memory leak and performance issue?

  • Yes
  • NO

Reviewer Checklist

  • All provided information are reviewed and ensured.

Copy link
Collaborator

@BalajiSGrid BalajiSGrid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes are fine

@BalajiSGrid BalajiSGrid merged commit 1c13f03 into EJ2-981297-Emp-Management Dec 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants