fwiw, i went to your site to check out your demo and it doesn't work. well, it might work but since you only have one newspaper listed in your db as far as i can tell (you only have one paper listed in the dropdown when 'show all' is selected), it doesn't really show that the module is robust to do that main thing i'd be interested in which is a proximity search based on a zip code.

that's not to say that your module isn't good or doesn't work as advertised, but it would help others if your demo represented a real life case that actually displays the 'proximity' functionality based on a zip code entered into the form. mostly because that's the beauty of what your module offers.

fyi, the feedback is not meant to incite but rather help you improve a module that might have some potential.

without the mods suggested above, i have to download and install the module and hope it's going to do what you say it does. that wastes time if it doesn't. with the plethora of drupal modules that exist today, it is quite fair to say that many are not as advertised. having a more robust demo serves the community better.

thanks for your efforts regardless

Comments

seanr’s picture

Status: Active » Postponed

Thanks for the tip. I'll get that list filled out later, but it's not a high priority at the moment. We're going to do a refresh of all the content on that demo site soon (including migrating to Drupal 6), so it'll probably get updated then.

nipsy’s picture

I think part of it might be a bug in the code.

The idea of a Letters module is AWESOME, but in my module install, it did not make the table correctly.

in the field "newsid" is says 'not null' and 'default 0' resulting in "duplicate" errors when trying to add more editors.

I changed it to "auto increment" and things are working much better.

seanr’s picture

Status: Postponed » Closed (fixed)

nipscy, that's a completely separate issue. Please create a new issue rather than tacking your bug onto an unrelated one.