Skip to content

Reimplemented delete reason modal in Request Details tabn modal#1313

Open
rupasree-guttikonda wants to merge 31 commits into
testfrom
feature/delete-reason-modal
Open

Reimplemented delete reason modal in Request Details tabn modal#1313
rupasree-guttikonda wants to merge 31 commits into
testfrom
feature/delete-reason-modal

Conversation

@rupasree-guttikonda
Copy link
Copy Markdown

This PR reimplements the delete request functionality with a reason modal in RequestDescription.jsx.

The previous implementation unintentionally removed some UI elements. This version ensures that all existing features and icons remain unchanged while adding the delete reason modal functionality.

Please review and let me know if any further changes are needed.

HemaVenkataSaiDumpala and others added 30 commits March 22, 2025 21:09
…and image from linkedin profile for new director.
merge latest main branch changes
Prod patch1 for updating executives on production website
@rupasree-guttikonda rupasree-guttikonda requested a review from a team as a code owner March 9, 2026 01:26
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 9, 2026

Deploy Preview for saayam-webapp-netlify-awa ready!

Name Link
🔨 Latest commit 52fffb2
🔍 Latest deploy log https://app.netlify.com/projects/saayam-webapp-netlify-awa/deploys/69ae2319d061d50008124e5e
😎 Deploy Preview https://deploy-preview-1313--saayam-webapp-netlify-awa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 9, 2026

Deploy Preview for elegant-treacle-3e6cef ready!

Name Link
🔨 Latest commit 52fffb2
🔍 Latest deploy log https://app.netlify.com/projects/elegant-treacle-3e6cef/deploys/69ae231964b12600080234ed
😎 Deploy Preview https://deploy-preview-1313--elegant-treacle-3e6cef.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 9, 2026

Deploy Preview for musical-gnome-285736 ready!

Name Link
🔨 Latest commit 52fffb2
🔍 Latest deploy log https://app.netlify.com/projects/musical-gnome-285736/deploys/69ae23196aca0600081f6ffd
😎 Deploy Preview https://deploy-preview-1313--musical-gnome-285736.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rupasree-guttikonda rupasree-guttikonda force-pushed the feature/delete-reason-modal branch from dbe1ad8 to 52fffb2 Compare March 9, 2026 01:32
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 9, 2026

Codecov Report

❌ Patch coverage is 15.58753% with 352 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/common/components/VoiceRecordingComponent.jsx 0.00% 207 Missing ⚠️
src/common/components/Navbar/Navbar.jsx 28.57% 95 Missing ⚠️
.../common/components/PhoneNumberInputWithCountry.jsx 0.00% 28 Missing ⚠️
src/common/components/DataTable/Table.jsx 59.45% 14 Missing and 1 partial ⚠️
.../common/components/RequestButton/RequestButton.jsx 0.00% 3 Missing ⚠️
src/common/components/Tabs/Tabs.jsx 71.42% 2 Missing ⚠️
src/common/components/Wizard/StepperControl.jsx 0.00% 2 Missing ⚠️

❌ Your patch check has failed because the patch coverage (15.58%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Files with missing lines Coverage Δ
src/Layout/Layout.jsx 100.00% <ø> (ø)
src/common/components/Navbar/Navbarold.jsx 0.00% <ø> (ø)
src/common/i18n/i18n.js 0.00% <ø> (ø)
src/common/i18n/languagesData.js 100.00% <ø> (ø)
src/common/i18n/localeList.js 0.00% <ø> (ø)
src/common/components/Tabs/Tabs.jsx 86.66% <71.42%> (ø)
src/common/components/Wizard/StepperControl.jsx 0.00% <0.00%> (ø)
.../common/components/RequestButton/RequestButton.jsx 0.00% <0.00%> (ø)
src/common/components/DataTable/Table.jsx 71.15% <59.45%> (ø)
.../common/components/PhoneNumberInputWithCountry.jsx 0.00% <0.00%> (ø)
... and 2 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rashmi-bilaskar
Copy link
Copy Markdown
Contributor

Please raise PR for test branch.

@rupasree-guttikonda rupasree-guttikonda changed the base branch from main to test March 20, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants