KartulUdus / PoracleJS

NodeJS application for PokémonGo WebHook Discord alarms
https://kartuludus.github.io/PoracleJS/#/
ISC License
96 stars 115 forks source link

a little Poracle spring update #906

Closed ReuschelCGN closed 7 months ago

ReuschelCGN commented 7 months ago
kamieniarz commented 7 months ago

Would it be possible to also add hook that could display reward(s)?

kamieniarz commented 7 months ago

@ReuschelCGN looks like "disableNotConfirmedInvasion": true fully turns off kecleon, goldstop and showcases alerts 😬

kamieniarz commented 7 months ago

Also something is broken with reactmap's poracle integration. When I open webhook management my gym and raid tabs are empty and also when I try to add gym tracking from map I'm getting poracle error:

2024-03-30 17:30:10 $MAIN error: API: 127.0.0.1 POST /api/tracking/gym/:id scannerQuery.getGymName is not a function {
  stack: 'TypeError: scannerQuery.getGymName is not a function\n' +
    '    at Object.gymRowText (/home/toaster/PoracleJS/src/lib/PoracleJSMessage/commands/tracked.js:82:66)\n' +
    '    at Object.<anonymous> (/home/toaster/PoracleJS/src/routes/apiTrackingGym.js:149:83)\n' +
    '    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)',
  [Symbol(level)]: 'error',
  [Symbol(splat)]: [
    TypeError: scannerQuery.getGymName is not a function
        at Object.gymRowText (/home/toaster/PoracleJS/src/lib/PoracleJSMessage/commands/tracked.js:82:66)
        at Object.<anonymous> (/home/toaster/PoracleJS/src/routes/apiTrackingGym.js:149:83)
        at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  ],
  [Symbol(message)]: '{"level":"error","message":"API: 127.0.0.1 POST /api/tracking/gym/:id scannerQuery.getGymName is not a function","stack":"TypeError: scannerQuery.getGymName is not a function\\n    at Object.gymRowText (/home/toaster/PoracleJS/src/lib/PoracleJSMessage/commands/tracked.js:82:66)\\n    at Object.<anonymous> (/home/toaster/PoracleJS/src/routes/apiTrackingGym.js:149:83)\\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)"}'
}

EDIT: looks like there's an issue with "scannerType" - I changet that from rdm to golbat and it broke

ReuschelCGN commented 7 months ago

broken reactmap's poracle integration should be fixed

ReuschelCGN commented 7 months ago

looks like "disableNotConfirmedInvasion": true fully turns off kecleon, goldstop and showcases alerts 😬

should be fixed now, sorry for that

Fabio1988 commented 7 months ago

Thanks a lot for that update :)

@jfberry you think it would be time to introduce some new webhook types? Different ones for Incident (kecleon, showcase) and invasion... A new one for confirmed lineups? Therefore we would prevent additional configs and blocking of webhooks

jfberry commented 7 months ago

Thanks a lot for that update :)

@jfberry you think it would be time to introduce some new webhook types? Different ones for Incident (kecleon, showcase) and invasion... A new one for confirmed lineups? Therefore we would prevent additional configs and blocking of webhooks

You'd still want these configs. Webhooks should not be blocked on the golbat side. Open to changes but not sure what you imagine would be different in terms of flow?

Fabio1988 commented 7 months ago

Thanks a lot for that update :) @jfberry you think it would be time to introduce some new webhook types? Different ones for Incident (kecleon, showcase) and invasion... A new one for confirmed lineups? Therefore we would prevent additional configs and blocking of webhooks

You'd still want these configs. Webhooks should not be blocked on the golbat side. Open to changes but not sure what you imagine would be different in terms of flow?

True config is still needed for the flow. I won't block in golbat as well... But it's kinda weird that this PR won't send hooks to discord for fort only areas anymore.

jfberry commented 7 months ago

Thanks a lot for that update :) @jfberry you think it would be time to introduce some new webhook types? Different ones for Incident (kecleon, showcase) and invasion... A new one for confirmed lineups? Therefore we would prevent additional configs and blocking of webhooks

You'd still want these configs. Webhooks should not be blocked on the golbat side. Open to changes but not sure what you imagine would be different in terms of flow?

True config is still needed for the flow. I won't block in golbat as well... But it's kinda weird that this PR won't send hooks to discord for fort only areas anymore.

Thats a problem with the implementation rather than the hooks though - it's a good point which may require some further consideration

jfberry commented 7 months ago

Are you going to address the config titles so we can get this merged?

jfberry commented 7 months ago

Ready to merge? I'm not sure the issue of non-lineup invasion types is resolved?