Support ATMs and WTT from Unified Money

minetest-mapserver / mapserver

Minetest realtime mapserver, written in go

Other

101 stars 22 forks source link

Support ATMs and WTT from Unified Money #372

Closed Emojigit closed 5 months ago

Emojigit commented 5 months ago

This PR adds ATMs and WTTs for Unified Money into the map server.

For some reason, the original code did not return L.icon(...). This PR added it back, fixing the display.

This is one of the ATMs on my server. Don't forget to enable the ATM layer first - it is not on by default.

Emojigit commented 5 months ago

The ATM is included in map object responses, but not on the map.

[
  {
    "mapblock": {
      "x": -100,
      "y": 0,
      "z": 34
    },
    "x": -1588,
    "y": 4,
    "z": 545,
    "type": "atm",
    "mtime": 1717542755,
    "attributes": {
      "type": "atm2"
    }
  }
]

coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 9382223698

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.636%

Totals
Change from base Build 7357732829:	0.0%
Covered Lines:	907
Relevant Lines:	1341

💛 - Coveralls

BuckarooBanzay commented 5 months ago

looks good, thanks :+1: