Skip to content

Add AvANa to directory.json - #346

Merged
rnestler merged 2 commits into
SpaceApi:masterfrom
boyska:patch-1
Oct 9, 2026
Merged

rnestler merged 2 commits into
SpaceApi:masterfrom
boyska:patch-1

Conversation

@boyska

@boyska boyska commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

If you're adding a new endpoint make sure that

  • The new entry is at the correct (alphabetically sorted) line.
  • The endpoint is valid according to https://validator.spaceapi.io/ui — it shows an error about the address not being valid, but it's just that indeed the hacklab is in such a place which doesn't have a "proper" address :)

@rnestler rnestler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Contrary to your declaration it fails both the sorting check and the validation check:

--- directory.json
+++ directory.json
@@ -9,8 +9,8 @@
   "Amman Valley MakerSpace": "https://raw.githubusercontent.com/AmmanVMS/space.api/main/api.json",
   "Apollo-NG": "https://apollo.open-resource.org/status.php",
   "Arch Reactor": "https://archreactor.org/status.php",
-  "AvANa": "https://avana.forteprenestino.net/spaceapi.json",
   "Attraktor Makerspace": "http://blog.attraktor.org/spaceapi/spaceapi.json",
+  "AvANa": "https://avana.forteprenestino.net/spaceapi.json",
   "AwesomeSpace": "https://state.awesomespace.nl/",
   "B4CKSP4CE": "https://spaceapi.0x08.in/",
   "Bastli": "https://bastli.ethz.ch/spaceapi.json",

validation:
Image

For the validity: 0.13 required to have the state field always present. By just declaring
"api_compatibility": ["14", "15"], and removing "api": "0.13", it should be fixed.

@boyska

boyska commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Contrary to your declaration it fails both the sorting check and the validation check:

ooops, fixed

validation: Image

For the validity: 0.13 required to have the state field always present. By just declaring "api_compatibility": ["14", "15"], and removing "api": "0.13", it should be fixed.

thanks for letting me understand what was the problem! I assumed it was something about the Geo coordinates, because that was the most prominent warning I could understand (and it is irrelevant in our case).

now everything should be fixed.

@boyska
boyska requested a review from rnestler October 8, 2026 12:14
@rnestler

rnestler commented Oct 9, 2026

Copy link
Copy Markdown
Member

thanks for letting me understand what was the problem! I assumed it was something about the Geo coordinates, because that was the most prominent warning I could understand (and it is irrelevant in our case).

Yeah missing keys are handled very badly by the validator UI: They are just not shown, since it tries to show error messages next to the key they affect (which doesn't exist if the key is missing). I needed to dig into the browser dev tools and look at the validator API response.

@rnestler
rnestler merged commit ff8c86f into SpaceApi:master Oct 9, 2026
2 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.

2 participants