issue_comments
Data license: AGPL · Data source: ACEmulator Project
4 rows where issue = 446766312
This data as json, CSV (advanced)
Suggested facets: user, author_association, created_at (date), updated_at (date)
id ▼ | html_url | issue_url | node_id | user | created_at | updated_at | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
494550040 | https://github.com/ACEmulator/ACE/issues/1922#issuecomment-494550040 | https://api.github.com/repos/ACEmulator/ACE/issues/1922 | MDEyOklzc3VlQ29tbWVudDQ5NDU1MDA0MA== | gmriggs 8909245 | 2019-05-21T20:41:07Z | 2019-05-21T20:52:11Z | COLLABORATOR | I agree in theory that the 'source' of the spell would be the item that cast it During initial dev/testing of ItemProc, this switched between item.TryCastSpell and wielder.TryCastSpell a few times. If I remember correctly, there were 1 or 2 bugs with item.TryCastSpell that need to be fixed One that I remember is the message that is broadcast to the target. It would say things like 'Sword cast spell on you' or 'Aetheria cast spell on you', which seemed very unclear (not sure how it was in retail). If the proc successfully landed, the wielder also would not receive any messages about it, without additional logic added I forget if that was the only thing that needed investigated, but I'm thinking there might also have been something with EnchantmentManager sources Anyway yeah this needs further investigated, just saying it's not as simple as changing wielder.TryCastSpell to TryCastSpell here, and to be aware of the other possible bugs that would need to be looked into for that | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Item Proc'ing - Needs to be corrected for Alchemical Throwning Phials to work 446766312 | |
494552419 | https://github.com/ACEmulator/ACE/issues/1922#issuecomment-494552419 | https://api.github.com/repos/ACEmulator/ACE/issues/1922 | MDEyOklzc3VlQ29tbWVudDQ5NDU1MjQxOQ== | gmriggs 8909245 | 2019-05-21T20:48:02Z | 2019-05-21T20:48:02Z | COLLABORATOR | One thing to possibly look out for with item.TryCastSpell / item sources is the enchantment duration If an item is the enchantment source, a lot of times the enchantment duration will be set to -1 I remember adding some special logic to work around this for Aetheria, and if all ItemProcs are casting the spell from item source, this part should be tested thoroughly | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Item Proc'ing - Needs to be corrected for Alchemical Throwning Phials to work 446766312 | |
496662382 | https://github.com/ACEmulator/ACE/issues/1922#issuecomment-496662382 | https://api.github.com/repos/ACEmulator/ACE/issues/1922 | MDEyOklzc3VlQ29tbWVudDQ5NjY2MjM4Mg== | ghost 10137 | 2019-05-28T19:48:49Z | 2019-05-28T19:48:49Z | NONE | Correct source and target routing and messaging will be needed for ProcSpell for the Alchemical Throwing Phials to work correctly that are to be added in content patch https://github.com/ACEmulator/ACE-World-16PY-Patches/pull/132 | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Item Proc'ing - Needs to be corrected for Alchemical Throwning Phials to work 446766312 | |
505396193 | https://github.com/ACEmulator/ACE/issues/1922#issuecomment-505396193 | https://api.github.com/repos/ACEmulator/ACE/issues/1922 | MDEyOklzc3VlQ29tbWVudDUwNTM5NjE5Mw== | gmriggs 8909245 | 2019-06-25T11:04:55Z | 2019-06-25T11:04:55Z | COLLABORATOR | Fixed in https://github.com/ACEmulator/ACE/pull/2043 | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | Item Proc'ing - Needs to be corrected for Alchemical Throwning Phials to work 446766312 |
Advanced export
JSON shape: default, array, newline-delimited, object
CREATE TABLE [issue_comments] ( [html_url] TEXT, [issue_url] TEXT, [id] INTEGER PRIMARY KEY, [node_id] TEXT, [user] INTEGER REFERENCES [users]([id]), [created_at] TEXT, [updated_at] TEXT, [author_association] TEXT, [body] TEXT, [reactions] TEXT, [performed_via_github_app] TEXT, [issue] INTEGER REFERENCES [issues]([id]) ); CREATE INDEX [idx_issue_comments_issue] ON [issue_comments] ([issue]); CREATE INDEX [idx_issue_comments_user] ON [issue_comments] ([user]);