| 08:02.13 | *** join/#arl Repo (~supybot@repos.curseforge.net) |
| 08:02.13 | *** mode/#arl [+v Repo] by ChanServ |
| 08:10.37 | *** join/#arl Repo (~supybot@repos.curseforge.net) |
| 08:10.37 | *** mode/#arl [+v Repo] by ChanServ |
| 08:23.14 | *** join/#arl pompy (~Mike@c-68-38-51-80.hsd1.nj.comcast.net) |
| 08:23.14 | *** mode/#arl [+o pompy] by ChanServ |
| 14:05.49 | *** join/#arl robotusch (~robotusch@50A2E67F.flatrate.dk) |
| 14:13.25 | Zhinjio | just a side note. I would recommend pushing a new beta since the current beta isn't working. |
| 14:13.27 | Zhinjio | or maybe you already have. |
| 14:14.28 | Zhinjio | yeah, you did. Thanks. |
| 14:28.04 | *** join/#arl Jim-Bim (~thorben_o@p54BB780B.dip.t-dialin.net) |
| 14:28.04 | *** mode/#arl [+o Jim-Bim] by ChanServ |
| 16:44.01 | *** join/#arl Thaoky (Thaoky@91.182.80.103) |
| 16:49.50 | pompy | Zhinjio: isnt working how? |
| 16:50.28 | Zhinjio | its fine. ignore me. |
| 16:50.34 | Zhinjio | the beta that was prior to 2 days ago was crashing wow |
| 16:50.58 | pompy | :O |
| 18:48.36 | Torhal | Yola folks. |
| 18:49.00 | Torhal | There has only been one beta thusfar. |
| 18:49.35 | Torhal | That should really be transparent. |
| 18:49.51 | Torhal | Having to go to a different AddOn to change Altoholic's settings is upsetting. |
| 18:50.01 | Thaoky | o/ |
| 18:50.10 | Thaoky | Well |
| 18:50.14 | Torhal | I wasn't happy that I couldn't turn off the mail expiry popup. |
| 18:50.25 | Torhal | Then pompy told me I had to do it from DataStore |
| 18:50.31 | Torhal | And I WTF'd |
| 18:50.37 | Thaoky | The thing is, it makes sense if you look at it from a "multiple client add-ons" perspective |
| 18:50.52 | Thaoky | But I admit that it may be misleading at first |
| 18:51.07 | Torhal | Not for the end-user. Someone downloading Altoholic expects to go into Altoholic's settings to change something. :D |
| 18:51.18 | Thaoky | I had an idea earlier this week |
| 18:51.18 | Torhal | Especially when the popup says "Altoholic: Mail..." |
| 18:51.20 | Thaoky | quite simple |
| 18:51.40 | Thaoky | change the altoholic + ds options buttons into drop down menus when you mouse over them |
| 18:51.55 | Thaoky | with direct links to each categories in the option panel |
| 18:53.48 | Torhal | Think of it this way: A web site changes from MySQL to PostreSQL - their users shouldn't notice nor care. They shouldn't be required to know that the database has changed and to do something different in order to access something they were used to accessing in a simple manner before. |
| 18:55.56 | Thaoky | I fully agree, but the fact is, ds is intentionally a bit more than just the db repository :) |
| 18:56.17 | Thaoky | hence the various helper functions, option panes, etc.. |
| 18:56.28 | Thaoky | my idea was: |
| 18:56.54 | Torhal | Aye, but couldn't you simply hide DS from Altoholic users? Have the "Mail Expiry Notification" setting in Altoholic which, when toggled, directly changed the DS value? |
| 18:57.24 | Thaoky | If I want another addon to use one of the modules, then it shouldn't have to rewrite support for options (especially for options that deal with scanning and that must be in there) |
| 18:57.50 | Thaoky | I could do that, but then, same reply as one line above :) |
| 18:57.51 | Torhal | Not saying re-write - I'm saying there should be an API |
| 18:58.00 | Torhal | Eh |
| 18:58.59 | Torhal | Well, here's what I'm saying: I didn't expect to have to go into DataStore to change a popup that said "Altoholic: Mail is about to expire on..." |
| 18:59.23 | Torhal | When I didn't see the option in Altoholic itself, I almost disabled it. Doing a reloadUI and getting a popup every time with no way to disable it is annoying |
| 19:00.10 | Torhal | pompy told me what I needed to do, so it isn't an issue any longer, but it just seemed...ugh from a User Experience standpoint |
| 19:00.45 | Thaoky | Agreed |
| 19:01.24 | Thaoky | I had a hard time making up my mind on how to code this correctly to begin with |
| 19:02.18 | Thaoky | Most likely, if I don't want to completely screw my approach, the best thing to do will be to bring this into DS_Mail completely (the dialog box I mean) |
| 19:02.24 | Thaoky | and change the text |
| 19:02.49 | Thaoky | with also a check box to "turn off this feature" right in the dialog box |
| 19:03.10 | Torhal | Another thing you could do for the mail expiry warnings is make the client AddOn request that information. That way individual AddOns which wished to support this could send a request to DS, and each AddOn could have it disabled as the user needed instead of going to DS and completely disabling it |
| 19:03.31 | Thaoky | In fact, it's sort of already this way |
| 19:03.36 | Torhal | Or have the popup say "DataStore: Mail is about to expire..." |
| 19:04.03 | Torhal | It doesn't make DataStore transparent to the user, but it does tell them where to go |
| 19:04.06 | Thaoky | DS Mails is checking for expiries no matter what, it simply does not send the warning to client addons if it's disabled |
| 19:04.46 | Torhal | I'm just somewhat uncomfortable with the notion of end-users being aware of the internal infrastructure of an AddOn :) |
| 19:04.50 | Torhal | They shouldn't need to. |
| 19:05.00 | Torhal | And most don't want to take the time to,. |
| 19:05.27 | Thaoky | Ideally, I'd hide that too, but I already had Odyssey in mind when I went from Altoholic's own DB to DS :) |
| 19:06.02 | Thaoky | And I didn't want to rewrite similar code, hence my decision to adopt a layered approach |
| 19:07.01 | Thaoky | For instance, when I wanted to add quest history tracking (for odyssey), it was basically done in an hour |
| 19:08.25 | Thaoky | Really, the best thing to do at this point is not to hide DS, but to make it gain visibility from Altoholic, and I think the drop down menus I intend to add should help |
| 19:09.24 | Thaoky | That is, if I don't want to roll back months of coding x) |
| 19:09.55 | Torhal | Hopefully :) |
| 19:10.59 | Thaoky | Anyway, as of next week, I should have more time to code.. so I'll fix a lot of stuff that have been left aside since a few weeks (months ? :x) |
| 19:12.53 | Thaoky | .. and thanks for your input too :) |
| 19:13.27 | Torhal | np :) |
| 20:10.39 | *** join/#arl Repo (~supybot@repos.curseforge.net) |
| 20:10.39 | *** mode/#arl [+v Repo] by ChanServ |
| 21:15.51 | *** part/#arl Thaoky (Thaoky@91.182.80.103) |