These are chat archives for arenanet/api-cdi

2nd
Feb 2017
Darrian
@rikkuness
Feb 02 2017 15:15
I know this is out of the remit of this room, but anyone happen to know a quick and easy IDP I can test SAML auth against? The one I've been provided in work doesn't seem to work and I need to prove it's not my code that's wrong haha
Darqam
@Darqam
Feb 02 2017 16:05
so about #435, is this live yet? My elementalist page doesn't seem to show the skills section
or maybe I'm blind
Darqam
@Darqam
Feb 02 2017 16:26
ok, I am dumb, it's still open
I think the backend support for that one goes out with the next release? Might be the one after.
I haven't been paying attention to the release schedule
ChieftainAlex
@ChieftainAlex
Feb 02 2017 19:05
black dye chatlink "can't link from location" ... has the chatlink blanket gone too far
lol
ChieftainAlex
@ChieftainAlex
Feb 02 2017 19:08
what's the rule that determines whether they're prevented from appearing?
just a range of ids?
darthmaim
@darthmaim
Feb 02 2017 19:08
you have to be underwater to link them
wait are you serious
darthmaim
@darthmaim
Feb 02 2017 19:08
lol
I have no idea where that error message comes from :/
ChieftainAlex
@ChieftainAlex
Feb 02 2017 19:09
someone clearly thought it would be a good idea to stop chatlink miners (lol) by preventing any links meeting certain criterion
however it seems a bit too strong..
Darqam
@Darqam
Feb 02 2017 19:10
so whatever changed, would be from around then
ChieftainAlex
@ChieftainAlex
Feb 02 2017 19:11
oh yeah lol, those naked outfit chatlinks
darthmaim
@darthmaim
Feb 02 2017 19:11
should use the api whitelist
Darqam
@Darqam
Feb 02 2017 19:11
And the amusing thing is that this new thing doesn't even stop all un-released chat codes
Jonathan Andrist
@rwfrk_twitter
Feb 02 2017 22:53
Serious miners go straight to the dat file. This just stops players from linking things they don't have on hand. The one thing I'd want that applied to would be Legendary Insights, but I haven't seen a single other item it makes sense for.
speaking of legendary insights, i'm waffling about the output format for #434
like right now I've got
const raids = {
    forsaken_thicket : {
        spirit_vale : {
            vale_guardian : true,
            // ... etc
        }
    }
}
but ... that doesn't really leave any room if we want to jam other things in there?
also I'm not entirely happy with the nesting bits
maybe /v2/raids is needed or something I dunno
Eearslya Sleiarion
@Eearslya
Feb 02 2017 22:58
Maybe instead of fixed object attributes, arrays with key-value objects?
Darqam
@Darqam
Feb 02 2017 22:58
you could also make something like an array filled with only the names of things killed
[ "vale_guardian", ... ] hmm
Darqam
@Darqam
Feb 02 2017 22:58
yeah
and then have /v2/raids/vale_guardian ?
there isn't a whole lot of metadata about raids that I can dump out easily
Darqam
@Darqam
Feb 02 2017 22:58
hum.... not sure what you would get from that endpoint
boss ordering is all I can think of
I can technically pull the mob/event names for localized strings?
(or just link the event directly)
oh right https://api.guildwars2.com/v2/events is dead, hmm
Darqam
@Darqam
Feb 02 2017 22:59
I thought the only point was to get a yes or no if boss was killed. I imagine if name is in array then it's killed, if not then it isn't.... but that isn't very "fancy"
chauncey would not approve
Eearslya Sleiarion
@Eearslya
Feb 02 2017 23:00
["forsaken_thicket":"name":"Forsaken Thicket","wings":[{"name":"Spirit Vale",bosses:[{"name":"Vale Guardian"}]}]]
Wait I messed that up
darthmaim
@darthmaim
Feb 02 2017 23:00
Why did it serve my half finished message...
because gitter :sadface:
darthmaim
@darthmaim
Feb 02 2017 23:01
Mobile gitter is so bad
I started writing and then clicked the issue link
Eearslya Sleiarion
@Eearslya
Feb 02 2017 23:02
[
  "forsaken_thicket": {
    "name": "Forsaken Thicket",
    "wings": [
      {"name": "Spirit Vale", bosses: [ { "name": "Vale Guardian" } ] }
    ]
  }
]
there
So the other question is -- does it make sense to mungle all the weekly counters into a single /v2/account/weeklies?
Eearslya Sleiarion
@Eearslya
Feb 02 2017 23:02
Maybe without the first key, just array of objects
Or should raids/dungeons be separate endpoints (/v2/account/raids, /v2/account/dungeons)
Darqam
@Darqam
Feb 02 2017 23:02
Well dungeons are dailies, so can't fit in a weekly thing
oh dang you're right.
darthmaim
@darthmaim
Feb 02 2017 23:05
account/progression (name?) with type = raid|dungeon, id and frequency = weekly|daily
Darqam
@Darqam
Feb 02 2017 23:05
I can't think of any other pertinent info you would throw in a "daily" endpoint. fotm are not capped beside daily achievs... which is tracked on its own. and daily capped crafts... well that seems silly to track that
"hey you haven't crafted your ascended things today!"
yeah.
darthmaim
@darthmaim
Feb 02 2017 23:06
And then normal bulk endpoints /v2/account/raids, /v2/account/dungeons to look them up
dungeons/raids don't really have ids -- it's kind of just a map with a pile of scripts
but I can just make up string ids
hmm
I'm pretty convinced about /v2/account/raids /v2/account/dungeons just returning boss/path ids for completed things, then having /v2/raids and /v2/dungeons if I find more metadata
the only caveat is that /v2/account/raids could return a bit more structure, e.g.
{
    "spirit_vale" : [ "vale_guardian", "sabetha" ]
}
I'm not sure if that's actually more useful than just ["vale_guardian", "sabetha"] though
got like a month before backend support for this lands, so plenty of time to waffle about it
darthmaim
@darthmaim
Feb 02 2017 23:09
But you would have to hardcode all available raids/.. then to show a list of unfinished ones
oh I'm going to have to hardcode them one way or another
darthmaim
@darthmaim
Feb 02 2017 23:10
You, yes
const raids = {
    forsaken_thicket : {
        spirit_vale : [
            {
                name : "vale_guardian",
                guid : "0ffde190-496b-401e-882a-ac8aac176be1"
            },
            // for pages
darthmaim
@darthmaim
Feb 02 2017 23:10
But app developers
If the API only returns completed
ah right I always forget about that
that'll not be an issue if /v2/raids existed with enough metadata to sort out the strings
darthmaim
@darthmaim
Feb 02 2017 23:11
Yes
Darqam
@Darqam
Feb 02 2017 23:12
not sure if it's important or not, but need to remember that we will keep getting new raid wings+encounters. So the structure should be adaptable to that (which the current examples seem to be anyway)
darthmaim
@darthmaim
Feb 02 2017 23:14
:+1: for localized bulk extended /v2/raids and authorized /v2/account/raids with just an array of ids
That would match other existing endpoints