I just checked your app for sql injection. The password I chose while registering a new list was d'artagnan and, sure enough, I was granted access without any error message whatsoever, but once I logged out I couldn't log in again. I thought that you might have removed the ' character automatically, but I also tried with dartagnan and couldn't log in either.
Ask HN: Please review my web app: WantsThis
21–29 of 29 posts
Re: Ask HN: Please review my web app: WantsThis
#22Re: Ask HN: Please review my web app: WantsThis
#23Hey, this looks great; I'm signing up purely out of interest. Though, one small thing - if neither password field is filled out and the form is sent, I receive this message: "The password fields must match". Insignificant non-bug, but still not the correct error message to display. I look forward to see where you go with this!
Re: Ask HN: Please review my web app: WantsThis
#24nice design and intuitive UI, I like it. What about sync the wish list instead of import the wish list? sync a wish list will cover import and export as well. As User, I will love this option, but sure there's a lot to do to make this work properly, if that's possible.
Re: Ask HN: Please review my web app: WantsThis
#25Re: Ask HN: Please review my web app: WantsThis
#26Earlier quoted context omitted.
I am seeing the FireFox issue, I think the CSS needs to be tweaked a little more. I am investigating the results for the search items now.. Thanks for the bug finding!
I like the bold styling but it's a bit too big and clunky IMO - you've got "Create your FREE wish list" then "Create Your Ultimate Wish List" then "Create the perfect wishlist for ..." all one after another. Instead I'd have this up next to your "WantsThis" banner with maybe one body text repitition above the fold, not 3. It's all a little bit too busy IMO. Also perhaps indicate on the direct purchase links (images i…
Re: Ask HN: Please review my web app: WantsThis
#27Re: Ask HN: Please review my web app: WantsThis
#28Re: Ask HN: Please review my web app: WantsThis
#29Easy to understand and use, I just don't see how this would be useful? Or why I would use it.