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

Use webpack5 syntax to include woff2 in bundle and remove url-loader #2217

Merged
merged 6 commits into from
Jul 26, 2024

Conversation

markmonster
Copy link
Contributor

What this PR does / why we need it:
This PR will fix the #2130 . The real developer of this PR is actually #2133 but this somehow got stuck in merging. That's why I tried to get the administrative process going after almost 5 months. The credits should go to schulz3000.

Which issue(s) this PR fixes: #2130

Special notes for your reviewer:
Use webpack5 syntax to include woff2 in bundle and remove url-loader that's no longer in use.

Does this PR introduce a user-facing change?:
It will reinstate the icons that got lost with a previous change.

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Unit tests passing
  • End-to-end tests passing

@github-actions github-actions bot added the UI label May 17, 2024
@markmonster
Copy link
Contributor Author

@dotnet-policy-service agree

@markmonster
Copy link
Contributor Author

@sungam3r do you know how I can ping the maintainer to look at this PR? Thanks.

* Bump from 1.10.4 to 1.12.0
@Jamie-Clayton
Copy link
Contributor

I've tested this solution locally and it resolved a range of issues that have been floated around v8.x of the health check UIs.

HealthChecks-EndToEnd-PR2217

LGTM

@Jamie-Clayton
Copy link
Contributor

@unaizorrilla - Would you have time to review some of these PR's, please? Gracias

@Jamie-Clayton
Copy link
Contributor

@Alirexaa this would be a good inclusion in v8.1

This was referenced Jun 21, 2024
@Alirexaa Alirexaa added this to the V 8.1 milestone Jun 22, 2024
---------

Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com>
@benjaminoerskov
Copy link

@markmonster Any idea when this can be released?

@markmonster
Copy link
Contributor Author

@markmonster Any idea when this can be released?

I have no idea at all. This is managed by the maintainers of this project. I hope it can be released soon.

@Alirexaa Alirexaa merged commit 73fdcc6 into Xabaril:master Jul 26, 2024
12 checks passed
@danmajgaard
Copy link

This was released by 2024-08-29 with AspNetCore.HealthChecks.UI version 8.0.2

(not by me, im just a user of the package :) )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants