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

Add spansh as a URL provider #2099

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

C1701D
Copy link
Member

@C1701D C1701D commented Nov 17, 2023

Description

This adds spansh as a URL provider for system and station URLs. It does so by adding a core plugin called spansh_core with said functionalities. The reason for that name is to make sure that custom plugins called spansh that users may already have installed don't result in either of these two plugins breaking. The spansh_core plugin is also designed to be expandable in the future if additional features should be added.

Closes #1490

Type of change

  • Adds a spansh plugin as a URL provider for systems and stations.

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

  • Multiple install runs.
  • Tested Plugin and core functionality in all relevant in-game situations.
  • All PyTest tests pass.

- Adds a spansh plugin as a URL provider for systems and stations.
- Adds plugin path to build.py.
- Updates a comment in the EDSM plugin.
@C1701D C1701D added enhancement Plugins Anything related to plugins labels Nov 17, 2023
@C1701D C1701D added this to the 5.10.0 milestone Nov 17, 2023
@C1701D C1701D self-assigned this Nov 17, 2023
@C1701D C1701D linked an issue Nov 17, 2023 that may be closed by this pull request
@Rixxan Rixxan self-requested a review November 17, 2023 21:17
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.

No blocking comments, but a few nits about type hinting and S/G

plugins/spansh_core.py Show resolved Hide resolved
plugins/spansh_core.py Show resolved Hide resolved
plugins/spansh_core.py Outdated Show resolved Hide resolved
plugins/spansh_core.py Outdated Show resolved Hide resolved
@Rixxan Rixxan merged commit 3e1725c into EDCD:develop Nov 17, 2023
1 check passed
@Rixxan Rixxan deleted the enhancement/1490/spansh-URL-provider branch November 17, 2023 21:43
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spansh: Implement URL providers
2 participants