pogenwurst wrote:There are a few issues:
pogenwurst wrote:- IMHO, the card is hard to edit. When I tried editing it myself, I went through a billion errors before I could get it right.
Yes, you have to do that in the source.
I'll see if I can change that.
pogenwurst wrote:- The link tag is messed up - I see the HTML when I receive the invitation. Do you have to configure the MIME type as text/html or something? (I really don't know)
No, you don't have to change the mime type. I thought some mail clients wouldn't interprete the link as a clickable one without the html tag but that was maybe wrong...
I'll fix that.
pogenwurst wrote:- The card should include some info on the board - title, admins, moderators, etc
Hm. I think I'll add some description of the board so people know *what* they are invited to. You have a point there.
But I think the admin names are not so important
pogenwurst wrote:- The admin plugin should have a field for a default header, footer, etc to send with each invite
- If you can get HTML working right, BBCode might be nice, though it wouldn't be that important
Sorry, I am very conservative in this and IMO emails have nothing, absolutely noting to do with HTML and they shouldn't. They are messages, information, whatever, and seeing the amount of mail traffic nowadays any tag that you do not send helps
So - no HTML mails for me, thanks.
pogenwurst wrote:I guess that's kind of a hefty list, but it's just my opinion. Overall, the mod is great. I'd just hate to see it spoiled by a few minor issues. I made a few minor chages to your code; if you want, I can email them to you, though I don't think they'd be much use, they're just aesthetic (did I spell that right?) changes.
Yes, go ahead, I am absolutely sure it is not perfect. This thing is terribly bad to test (no esxcuse, just an explanation )
pogenwurst wrote:Also, when someone doesn't have any invitations left, a message should be displayed in the profile view, rather than just a blank pane. I'm not sure - do new members automatically receive invites based on the amount given to each group in the plugin? If not, they should.
You are right there, however, this is difficult to implement.
I'll see for that when I find the time.