Skip to content
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.

Sign up for GitHub

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

Jump to bottom

1681 - feat(cms): add internal business project to case summary #1647

Closed
wants to merge 2 commits into from
Closed

1681 - feat(cms): add internal business project to case summary #1647

wants to merge 2 commits into from

Conversation

serenaabbott11
Copy link
Contributor

@serenaabbott11 serenaabbott11 commented Oct 27, 2023 •
edited
Loading

Changes in this PR

  • Add internal business project select field to the edit case summary page including the ability to add a new project to the existing projects and displays the project in the case summary
  • Disables the case source select field so agents cannot change the case source

Screen-shots or screen-capture of UI changes

Screenshot 2023-11-01 at 11 49 36

@serenaabbott11 serenaabbott11 added the enhancement New feature or request label Oct 27, 2023
@serenaabbott11 serenaabbott11 self-assigned this Oct 27, 2023
@viezly
Copy link

viezly bot commented Oct 27, 2023 •
edited
Loading

This pull request is split into 4 parts for easier review.
👀 Review pull request on Viezly

Changed files are located in these folders:

  • app
  • config/locales
  • db
  • spec/features/support

@serenaabbott11 serenaabbott11 force-pushed the pwnn-1681-internal-business-source branch 3 times, most recently from 2ae4719 to c2d9cc4 Compare November 1, 2023 13:13
@serenaabbott11 serenaabbott11 force-pushed the pwnn-1681-internal-business-source branch from c2d9cc4 to 43b529e Compare November 1, 2023 16:14
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Nov 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ryantk ryantk closed this Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryantk ryantk ryantk approved these changes

@EdwinKruglov EdwinKruglov Awaiting requested review from EdwinKruglov

Assignees

@serenaabbott11 serenaabbott11

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@serenaabbott11 @ryantk