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

🐛 On branch BUG/1608_glances-network-error #1646

Merged
merged 4 commits into from
Sep 1, 2024
Merged

🐛 On branch BUG/1608_glances-network-error #1646

merged 4 commits into from
Sep 1, 2024

Conversation

hockwill
Copy link
Contributor

@hockwill hockwill commented Jul 22, 2024

hockwill Quick hockwill /BUG/1608_glances-network-error → Lissy93/dashy Commits: 2 | Files Changed: 2 | Additions: 0 Label Powered by Pull Request Badge

Category:
BUGFIX

Overview
Glances Widgets for Network Traffic and Interfaces were not correctly displaying information provided from the api response.

Issue Number #1608

Code Quality Checklist (Please complete)

  • All changes are backwards compatible
  • All lint checks and tests are passing
  • There are no (new) build warnings or errors
  • [NA] (If a new config option is added) Attribute is outlined in the schema and documented
  • [NA] (If a new dependency is added) Package is essential, and has been checked out for security or performance
  • [NA] (If significant change) Bumps version in package.json

hockwill added 2 commits July 21, 2024 22:38
 Changes to be committed:
	modified:   src/components/Widgets/GlNetworkInterfaces.vue
	modified:   src/components/Widgets/GlNetworkTraffic.vue

bug ref: #1608

v4api ref: https://glances.readthedocs.io/en/latest/api.html#get-network
v3api ref: https://github.com/nicolargo/glances/blob/support/glancesv3/docs/api.rst#get-network
@hockwill hockwill requested a review from Lissy93 as a code owner July 22, 2024 02:57
Copy link

netlify bot commented Jul 22, 2024

Deploy Preview for dashy-dev ready!

Name Link
🔨 Latest commit 12e3dbe
🔍 Latest deploy log https://app.netlify.com/sites/dashy-dev/deploys/66c9b9b1eac958000896dc29
😎 Deploy Preview https://deploy-preview-1646--dashy-dev.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.

Copy link
Collaborator

@CrazyWolf13 CrazyWolf13 left a comment

Choose a reason for hiding this comment

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

Awesome, just tested and all plugins except the IP plugin seems to work, (the IP plugin shows a message that the glances version is not compatible with that plugin)

Thanks for your contribution! ✨

@CrazyWolf13
Copy link
Collaborator

Fixes #1608 and #1594 🚀

@hockwill
Copy link
Contributor Author

hockwill commented Aug 7, 2024 via email

@hockwill
Copy link
Contributor Author

hockwill commented Aug 9, 2024

Awesome, just tested and all plugins except the IP plugin seems to work, (the IP plugin shows a message that the glances version is not compatible with that plugin)

The glances gl-ip-address widget seems to work with no errors? Where is the version error message?

Copy link
Owner

@Lissy93 Lissy93 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @hockwill and @CrazyWolf13

@hockwill
Copy link
Contributor Author

I see this was approved a bit ago. Is there something else I need to do that I have not done?
Sorry for the close/open. I was seeing if it was something I needed to press.

@CrazyWolf13
Copy link
Collaborator

Yes, a merge was needed :).

@CrazyWolf13 CrazyWolf13 merged commit bede08c into Lissy93:master Sep 1, 2024
8 checks passed
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