Live data from Hacker News

Ask HN: Please review my web app: WantsThis

news.ycombinator.com

21–29 of 29 posts

Re: Ask HN: Please review my web app: WantsThis

#21
post #19

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.

You should be able to use an apostrophe in a password without issue as we did test for that. I'll see if I can reproduce that error.

Re: Ask HN: Please review my web app: WantsThis

#22
nice 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

#23

Hey, 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!

Fix is going up for this in the next few minutes, thanks!

Re: Ask HN: Please review my web app: WantsThis

#24

nice 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.

I think I saw this on another site, I'll see if their API allows for deleting and adding items externally.

Re: Ask HN: Please review my web app: WantsThis

#26

Earlier 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…

Excellent suggestion, other people have also told me the front page needs a bit of work and I agree. Thanks for the input
Post reply on HN