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.

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

fix(notifications): parse detail for notification body #1192

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

mwangggg
Copy link
Member

@mwangggg mwangggg commented Dec 24, 2023

Welcome to Cryostat! 👋

Before contributing, make sure you have:

  • Read the contributing guidelines
  • Linked a relevant issue which this PR resolves
  • Linked any other relevant issues, PR's, or documentation, if any
  • Resolved all conflicts, if any
  • Rebased your branch PR on top of the latest upstream main branch
  • Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
  • Signed all commits using a GPG signature

To recreate commits with GPG signature git fetch upstream && git rebase --force --gpg-sign upstream/main


Fixes: #1120
Depends on: cryostatio/cryostat#260

@mwangggg mwangggg force-pushed the 1120-error-response-cleanup branch 2 times, most recently from 54c6456 to 9f7c7b6 Compare January 16, 2024 17:01
@mwangggg mwangggg force-pushed the 1120-error-response-cleanup branch from 6730633 to 2a3879e Compare January 16, 2024 18:32
@mwangggg mwangggg added the dependencies Pull requests that update a dependency file label Jan 23, 2024
@andrewazores andrewazores added dependent and removed dependencies Pull requests that update a dependency file labels Feb 6, 2024
@andrewazores
Copy link
Member

/build_test

@andrewazores
Copy link
Member

CI failure due to -core not being buildable because it's still trying to pull and build an old version that has the broken jfrog repository configuration.

I'm trying to get -core 2.28.0 onto Maven Central now. If/when that works and we update all our projects to that dependency version then we should be finally out of the woods with this GitHub Packages Maven mess and not need authentication to download the dependency, or need to rebuild it like this in other project dependency chains, etc.

@andrewazores
Copy link
Member

@andrewazores
Copy link
Member

/build_test

@andrewazores
Copy link
Member

andrewazores commented Feb 6, 2024

Copy link

github-actions bot commented Feb 6, 2024

Test image available:

CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-web:pr-1192-2a3879e87deca75aa02d706daf2428db5c9ed9f9 bash smoketest.bash # then open http://localhost:8080

@andrewazores andrewazores force-pushed the 1120-error-response-cleanup branch from 2a3879e to fe45815 Compare February 6, 2024 23:03
@github-actions github-actions bot removed the dependent label Feb 6, 2024
@andrewazores andrewazores force-pushed the 1120-error-response-cleanup branch from fe45815 to 87c90f1 Compare February 6, 2024 23:04
@andrewazores andrewazores merged commit f870683 into cryostatio:main Feb 6, 2024
18 checks passed
@mwangggg mwangggg deleted the 1120-error-response-cleanup branch March 12, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Raw HTTP error response displayed when uploading actions fail
2 participants