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

do not load non-existant charts.js #63

Merged
merged 1 commit into from
Aug 24, 2023
Merged

do not load non-existant charts.js #63

merged 1 commit into from
Aug 24, 2023

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented May 22, 2023

We used charts.js from Foreman, but that was dropped in Foreman 3.0 (theforeman/foreman@69379b5) and results in 404 errors when trying to load some pages

Fixes: 28d49d8

@evgeni
Copy link
Member Author

evgeni commented Aug 17, 2023

👀 @laugmanuel @kamils-iRonin ;-)

@laugmanuel
Copy link
Member

Can you retrigger Github Actions? I'm not that familiar with the plugin. Can you jump in @kamils-iRonin ?

@evgeni
Copy link
Member Author

evgeni commented Aug 22, 2023

for some reason I cannot, but I can force push?

@evgeni
Copy link
Member Author

evgeni commented Aug 22, 2023

doesn't look any happier 🤷‍♀️

@kamils-iRonin
Copy link
Member

I opened a new PR (#65) to fix the pipeline, I don't think rubocop failure is related to this one

We used charts.js from Foreman, but that was dropped in Foreman 3.0
(theforeman/foreman@69379b5)
and results in 404 errors when trying to load some pages

Fixes: 28d49d8
@evgeni
Copy link
Member Author

evgeni commented Aug 23, 2023

all 🍏

@evgeni evgeni merged commit ca48aa8 into master Aug 24, 2023
11 checks passed
@evgeni evgeni deleted the top-of-the-patches branch August 24, 2023 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants