-
Notifications
You must be signed in to change notification settings - Fork 107
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
EPMRPP-87316 || Implement functionality to add Assignee and Milestone… #3644
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3644 +/- ##
========================================
Coverage 60.21% 60.21%
========================================
Files 79 79
Lines 857 857
Branches 124 124
========================================
Hits 516 516
Misses 314 314
Partials 27 27 ☔ View full report in Codecov by Sentry. |
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.
app/src/componentLibrary/autocompletes/singleAutocomplete/singleAutocomplete.jsx
Outdated
Show resolved
Hide resolved
… for GitLab issue by typing its name EPMRPP-87317 || Implement functionality to add Epic and Assignees for GitLab issue by typing its name
… for GitLab issue by typing its name EPMRPP-87317 || Implement functionality to add Epic and Assignees for GitLab issue by typing its name
69ce3a2
to
6abd271
Compare
...components/integrations/elements/bts/btsPropertiesForIssueForm/btsPropertiesForIssueForm.jsx
Outdated
Show resolved
Hide resolved
app/src/components/fields/dynamicFieldsSection/fields/multipleAutocompleteField.jsx
Outdated
Show resolved
Hide resolved
app/src/components/fields/dynamicFieldsSection/fields/autocompleteField.jsx
Outdated
Show resolved
Hide resolved
app/src/componentLibrary/autocompletes/common/autocompleteMenu/autocompleteMenu.scss
Outdated
Show resolved
Hide resolved
app/src/componentLibrary/autocompletes/singleAutocomplete/singleAutocomplete.jsx
Outdated
Show resolved
Hide resolved
app/src/componentLibrary/autocompletes/singleAutocomplete/singleAutocomplete.jsx
Outdated
Show resolved
Hide resolved
app/src/componentLibrary/autocompletes/multipleAutocomplete/multipleAutocomplete.jsx
Outdated
Show resolved
Hide resolved
EPMRPP-87317 || code review fixes - 1
EPMRPP-87317 || code review fixes - 2
EPMRPP-87317 || code review fixes - 3
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
const value = (inputValue || '').trim(); | ||
const uri = getURI(value); | ||
const requestParams = getRequestParams(value); |
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.
As I can see, If getRequestParams is not provided, the result of this function invocation will be undefined
.
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.
Fixed in the scope of #3670
… for GitLab issue by typing its name
EPMRPP-87317 || Implement functionality to add Epic and Assignees for GitLab issue by typing its name