Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove Related Degree and User Feedback #984

Merged
merged 1 commit into from
Feb 14, 2025
Merged

Remove Related Degree and User Feedback #984

merged 1 commit into from
Feb 14, 2025

Conversation

annaibm
Copy link
Contributor

@annaibm annaibm commented Feb 14, 2025

  • Remove temporarily Related Degree and User Feedback on possible issue page

related: #978

@netlify Netlify
Copy link

netlify bot commented Feb 14, 2025 •
edited
Loading

✅ Deploy Preview for eclipsefdn-adoptium-trss ready!

Name Link
🔨 Latest commit 74281a2
🔍 Latest deploy log https://app.netlify.com/sites/eclipsefdn-adoptium-trss/deploys/67afc7ec1597f00008d6fd5d
😎 Deploy Preview https://deploy-preview-984--eclipsefdn-adoptium-trss.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 site configuration.

@annaibm
Copy link
Contributor Author

annaibm commented Feb 14, 2025

Screenshot 2025-02-14 at 4 42 13 PM

@llxia
Copy link
Contributor

llxia commented Feb 14, 2025

And also remove

aqa-test-tools/test-result-summary-client/src/Build/PossibleIssues.jsx

Lines 185 to 193 in 4e40e90

let relatedDegree = 'Medium';
if (repoName.includes(mlIssueRepo)) {
if (
relatedIssues.items[index].number.toString() ===
mlIssueNum
) {
relatedDegree = 'High';
}
}

@annaibm annaibm force-pushed the removePossibleIssues branch from 4e40e90 to eb98b43 Compare February 14, 2025 22:42
@annaibm
Remove Related Degree and User Feedback
74281a2 
- Remove  temporarily Related Degree and User Feedback on
possible issue page

related: adoptium#978
Signed-off-by: Anna Babu Palathingal <anna.bp@ibm.com>
@annaibm annaibm force-pushed the removePossibleIssues branch from eb98b43 to 74281a2 Compare February 14, 2025 22:47
@annaibm
Copy link
Contributor Author

annaibm commented Feb 14, 2025

Screenshot 2025-02-14 at 5 48 40 PM

@annaibm annaibm marked this pull request as ready for review February 14, 2025 22:49
llxia
llxia approved these changes Feb 14, 2025
View reviewed changes
Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

LGTM

@llxia llxia merged commit 2b86b66 into adoptium:master Feb 14, 2025
7 checks passed
@llxia llxia mentioned this pull request Feb 14, 2025
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@llxia llxia llxia approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@annaibm @llxia