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

Update the api and much more #302

Merged
merged 83 commits into from
Nov 17, 2023
Merged

Update the api and much more #302

merged 83 commits into from
Nov 17, 2023

Conversation

Thorinwasher
Copy link

Refactor of the api necessary to facilitate more advanced interaction between modules and portals.

  • Add interface for registiring/unregestiring portal positions protected by stargate
  • Add interface for removing/adding portals
  • Add builders for portals and gates
  • Add interface for creating networks
  • Modify source code to be able to do the above changes

For example BlockInterfaceHandler and a Priority enum
Adds unittest to test that (Does not pass at the moment)
Probably needs some testing as well.

Also split PortalData into two different classes
@Thorinwasher Thorinwasher linked an issue Nov 17, 2023 that may be closed by this pull request
6 tasks
@ghost

This comment was marked as resolved.

Copy link

sonarqubecloud bot commented Nov 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 50 Code Smells

No Coverage information No Coverage information
0.4% 0.4% Duplication

@Pheotis Pheotis merged commit f466aaa into nightly Nov 17, 2023
3 of 4 checks passed
@Pheotis Pheotis deleted the apiv2 branch November 17, 2023 18:56
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.

The API Update
2 participants