Closed therebelrobot closed 8 years ago
@therebelrobot thanks for the PR, that's a great idea.
Btw @therebelrobot I'm not really sure why we have a field named video when it's actually for image-url so it'll probably be changed at some point but we'll keep your screenshot as the routeParam.
Example project url:
http://localhost:9000/add/project?name=Test&description=cool%20description&screenshot=screenshot-url&link=website-url&tags=this,that,foo,bar
Example tool url:
http://localhost:9000/add/tool?name=Test&description=cool%20description&avatar=avatar-link&link=website-link&license=Open&pricing=Free&twitter=@example&screenshot=screenshot-url&tags=this,that,foo,bar
See #193
The only major change I made to the variable nomenclature is the replacement of
video
andimage
withscreenshot
, in tools and projects respectively. This shouldn't effect how the form works internally, I just wanted to standardize the name between the two for future work on a bookmarklet.