Webtools:Mozparty: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Fixes from Ryan on the last two known issues)
 
(12 intermediate revisions by 5 users not shown)
Line 1: Line 1:
= Code =
= Project Information =
* [http://lxr.mozilla.org/mozilla/source/webtools/partytool PartyTool CVS]
* [http://wiki.mozilla.org/Webtools:Mozparty/Meetings Meeting Notes]
* [http://lxr.mozilla.org/mozilla/source/webtools/partytool/README README]
*'''Repository:''' http://viewvc.svn.mozilla.org/vc/projects/mozparty/trunk/
*'''Development Version:''' 2.0
*'''Production Site:''' http://www.firefoxparty.com
*'''Stage:''' https://firefoxparty.stage.mozilla.com
*'''Bugs:''' [https://bugzilla.mozilla.org/buglist.cgi?query_format=advanced&short_desc_type=allwordssubstr&short_desc=&product=Webtools&component=Party+Tool&long_desc_type=substring&long_desc=&bug_file_loc_type=allwordssubstr&bug_file_loc=&status_whiteboard_type=allwordssubstr&status_whiteboard=&keywords_type=allwords&keywords=&bug_status=NEW&bug_status=ASSIGNED&resolution=---&emailassigned_to1=1&emailtype1=exact&email1=&emailassigned_to2=1&emailreporter2=1&emailqa_contact2=1&emailtype2=exact&email2=&bugidtype=include&bug_id=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&cmdtype=doit&order=Reuse+same+sort+as+last+time&field0-0-0=noop&type0-0-0=noop&value0-0-0= Open] [https://bugzilla.mozilla.org/enter_bug.cgi?product=Webtools&component=Party%20Tool&op_sys=all&rep_platform=all File a new bug]
*'''Development Contact:''' [[User:Rflint]]


= Notes About Issues =
= Links =
* This was a good first-run -- this is meant as constructive feedback, and we hope this helps.  We appreciate the time you spent on this.  :)
* [[Webtools:Mozparty/L10n|L10n]]
* Before pushing this into production, I suggest fixing all remaining issues below.  I don't see any "eh, it's okay" issues -- they are all pretty core problems since the app is centered around users and invites and those two actions have either extra or unknown steps without good UI flow.
* [[Webtools:Mozparty/Release:2.2|2.2 release schedule]]
* I don't think any of the issues below are show stoppers - lets get this staged and get some more people playing around with it.  Our timeline is shrinking fast.--[[User:clouserw]]


= New Issues =
= Installation =
* Migration plans from [http://firefoxparty.com/ current party tool] need to be worked out
== 1.0 ==
** All users currently stored in flat-files for every country
* [http://lxr.mozilla.org/mozilla/source/webtools/partytool/README README]
* README with steps to test (verify that things are working) -- there isn't a solid way to test all of this -- so even after it's set up, what are the criteria to test it by to make sure it works end-to-end?
== 2.0 ==
 
* TBW
 
== Requirements ==
== Wil's Points ==
=== 1.0 ===
* <strike>/config/database.php shouldn't be in the repository. IT prefers a .default or -dist file.  (Same goes for config/bootstrap.php if you expect IT to change values in it)</strike>
* Apache 1.3+ (with mod_rewrite)
 
* PHP 4.3+
* <strike>There needs to be a README with
* A MySQL or PostgreSQL database
** description of project
* [http://www.cakephp.org/ CakePHP] 1.1.7.x+
** any special server requirements
=== 2.0 ===
** steps to install</strike>
* Apache 1.3+ (with mod_rewrite)
 
* PHP 5.1+
* <strike>Line 288 of controllers/user_controller.php has a hardcoded link in it that isn't going to work.</strike>
* MySQL
 
* <strike>Going to /user/activate/<random characters> inserts new rows in the database</strike>
 
* <strike>'''When updating a profile, if the passwords don't match the error just says "There was an error in your submission..." but doesn't say what it was''' -- There is still a problem here. There is a JS alert, then it'll just change it without any error now.</strike> - fixed (as well as the other validation issues below). There was an invalidate() missing.
 
* <strike>When updating a profile, if input is "<'name" it is saved/returned as "&lt;&#39;name". Each time the profile is updated, the entities are re-encoded again.</strike>
 
* <strike>When I created the party I did not choose "Invite only," however, when I go to the "Invite a guest" option it says "Since your party is invite only..."</strike>
 
* <strike>/register/ is a 404, but is linked to in the code (line 333 in controllers/party_controller.php - might be elsewhere too)</strike> -- haven't seen this.  An invite code might be null and cause this in the controllers, but I didn't produce that situation
 
* <strike>After updating a profile, the user is logged out</strike>
 
* <strike>When updating a profile, if a password is entered into the first box but not the second (or the second doesn't match), the password overwrites the saved password in the database in plain text.</strike>
 
* <strike>When updating a profile, if a password is entered into the second box but not the first, the form continues submission, no updates are made, no errors are shown</strike>
 
* <strike>going to /party/view/<number> and clicking "Count me in!" will create rows in the database for parties that don't exist</strike>
 
* <strike>When viewing a party that is invitation only, there is no way for someone to tell that they have to be invited. -- This is still an issue -- a user really has no way of seeing "parties I've been invited to".</strike>
 
* <strike>After the invite is sent out, user goes to register page, but then has no where to go. The original invite is lost, somehow. Also, if you send an invite out to an existing user, why shouldn't have to re-register, and they should also retain the invite context (be added to the party when they do log in).</strike>
* <strike>Session method should be 'database', not 'php'. If this were deployed on a cluster we'd want to avoid using /tmp for storing session data.  See line: 78
78    define('CAKE_SESSION_SAVE', 'php');</strike>
 
* <strike>Line 46 of users_controller.php redirects to "user" which no longer exists - that should be "users"</strike>
 
* <strike>If you go to /parties/invite/$number you get a page that offers to invite guests with a URL or an email address.  The URL gives a 404, it should be s/users/parties/</strike>

Latest revision as of 00:29, 3 June 2008

Project Information

Links

Installation

1.0

2.0

  • TBW

Requirements

1.0

  • Apache 1.3+ (with mod_rewrite)
  • PHP 4.3+
  • A MySQL or PostgreSQL database
  • CakePHP 1.1.7.x+

2.0

  • Apache 1.3+ (with mod_rewrite)
  • PHP 5.1+
  • MySQL
Retrieved from "https://wiki.allizom.org/index.php?title=Webtools:Mozparty&oldid=95187"