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

Broken Plugins Popup #2110

Merged
merged 4 commits into from
Dec 15, 2023
Merged

Conversation

C1701D
Copy link
Member

@C1701D C1701D commented Dec 14, 2023

Description

Adds a popup at startup about plugins that failed to load, presumably due to a wrong folder structure. This popup will come up at every EDMC start, unlike the "Plugin without Python 3.x support" one that only pops up every 24h. This PR also adds a list of those Plugins under their own entry in the Settings - Plugins tab.

Type of change

  • Adds a Popup at startup about Plugins that failed to load.
  • Adds entry about broken Plugins in the Settings - Plugins tab.
  • Adds corresponding translation entries.
  • Includes a minor fix for try/except if coriolis-data is missing.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • All PyTest tests pass.
  • find_localised_strings.py doesn't complain.
  • Multiple install runs.
  • Tested feature with multiple "broken" plugins.

Phoebe and others added 3 commits December 14, 2023 01:06
- Adds a Popup at startup about Plugins that failed to load.
- Adds entry about broken Plugins in the Settings - Plugins tab.
- Adds corresponding translation entries.
Also includes a minor fix for try/except if coriolis-data is missing
@C1701D C1701D added enhancement Plugins Anything related to plugins ui User Interface labels Dec 14, 2023
@C1701D C1701D added this to the 5.10.0 milestone Dec 14, 2023
@C1701D C1701D requested a review from Rixxan December 14, 2023 00:51
@C1701D C1701D self-assigned this Dec 14, 2023
Forgot to update the comment to be about broken plugins instead of no 3.x support ones.
Copy link
Contributor

@Rixxan Rixxan left a comment

Choose a reason for hiding this comment

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

Changes look good and appear to operate as intended.

@Rixxan Rixxan merged commit a38cf5a into EDCD:develop Dec 15, 2023
1 check passed
@Rixxan Rixxan deleted the enhancement/broken-plugin-popup branch December 15, 2023 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Plugins Anything related to plugins ui User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants