Line 1: |
Line 1: |
| + | {{draft}} |
| + | |
| {{ GoogleTrans-en | es =show | bg =show | zh-CN =show | zh-TW =show | hr =show | cs =show | da =show | nl =show | fi =show | fr =show | de =show | el =show | hi =show | it =show | ja =show | ko =show | no =show | pl =show | pt =show | ro =show | ru =show | sv =show }} | | {{ GoogleTrans-en | es =show | bg =show | zh-CN =show | zh-TW =show | hr =show | cs =show | da =show | nl =show | fi =show | fr =show | de =show | el =show | hi =show | it =show | ja =show | ko =show | no =show | pl =show | pt =show | ro =show | ru =show | sv =show }} |
| {{TOCright}} | | {{TOCright}} |
Line 19: |
Line 21: |
| == Actions == | | == Actions == |
| === Push to Public === | | === Push to Public === |
− | If you are reviewing a pending update, pushing public will cause the sandboxed version of the public add-on to appear on the public side. | + | If you are reviewing a pending update, pushing public will cause the sandboxed version of the public activity to appear on the public side. |
| | | |
− | If you are reviewing a nominated add-on, pushing public will cause both the add-on and its most recent version to appear on the public side. | + | If you are reviewing a nominated activity, pushing public will cause both the activity and its most recent version to appear on the public side. |
| | | |
| Your action and comments entered will be e-mailed to the author. | | Your action and comments entered will be e-mailed to the author. |
Line 28: |
Line 30: |
| If you are reviewing a pending update, retaining in sandbox will keep the version in the sandbox. The author will have to submit another update to be reviewed again. | | If you are reviewing a pending update, retaining in sandbox will keep the version in the sandbox. The author will have to submit another update to be reviewed again. |
| | | |
− | If you are reviewing a nominated add-on, retaining in sandbox will keep the add-on in the sandbox. The author is able to immediately nominate it again. If the author abuses this and does not make any changes, please flag for super review and indicate the reason. | + | If you are reviewing a nominated activity, retaining in sandbox will keep the activity in the sandbox. The author is able to immediately nominate it again. If the author abuses this and does not make any changes, please flag for super review and indicate the reason. |
| | | |
| Your action and comments entered will be e-mailed to the author. | | Your action and comments entered will be e-mailed to the author. |
| | | |
| === Request Super-Review === | | === Request Super-Review === |
− | Requesting super review will cause the add-on to be flagged for admin review, which will appear in an admin queue and dispatch an e-mail to the administrators. | + | Requesting super review will cause the activity to be flagged for administrative review, which will appear in an admin queue and dispatch an e-mail to the administrators. |
| | | |
| Super review should be requested for the following reasons: | | Super review should be requested for the following reasons: |
| * Security concerns | | * Security concerns |
| * Copyright/trademark concerns | | * Copyright/trademark concerns |
− | * Repeatedly nominating add-on without any changes | + | * Repeatedly nominating activity without any changes |
− | * Add-on contains binary components | + | * Activity contains binary components |
| * Other issues that an administrator should look into | | * Other issues that an administrator should look into |
| | | |
Line 47: |
Line 49: |
| Remember that your comments go to a *real person*, so try to be friendly. | | Remember that your comments go to a *real person*, so try to be friendly. |
| | | |
− | If you are pushing the add-on public, thank the author for the time and effort they have put in. Remember that they're enhancing the usefulness and the appeal of the products! Tell them which features you like the most, and mention anywhere you think they could improve. | + | If you are pushing the activity public, thank the author for the time and effort they have put in. Remember that they're enhancing the usefulness and the appeal of the products! Tell them which features you like the most, and mention anywhere you think they could improve. |
| | | |
| If you are retaining in the sandbox, be polite in pointing out any problems. Provide suggestions as to what they could do or should fix before re-submitting it and encourage them to re-submit it if it could ever be public. It's also a good idea to tell the author what you liked about the extension/theme even though you've retained it. | | If you are retaining in the sandbox, be polite in pointing out any problems. Provide suggestions as to what they could do or should fix before re-submitting it and encourage them to re-submit it if it could ever be public. It's also a good idea to tell the author what you liked about the extension/theme even though you've retained it. |
| | | |
− | If you think it should be translated, you can add a hint for submitting it to [http://www.babelzilla.org BabelZilla], even if it has no locale structure. | + | If you think it should be translated, you can add a hint for submitting it to the Localization Team, even if it has no locale structure. |
− | | |
| | | |
| = Reviewing Activities = | | = Reviewing Activities = |
Line 60: |
Line 61: |
| == Step 1. Decide on Testing Scope == | | == Step 1. Decide on Testing Scope == |
| * If an activity is designated for Sugar but also can run outside of Sugar, usually, you only need to test it within Sugar | | * If an activity is designated for Sugar but also can run outside of Sugar, usually, you only need to test it within Sugar |
− | * If you have Sugar running on multiple platforms, it would be great if you could do some testing on both platforms. | + | * If you have Sugar running on multiple platforms, it would be great if you could do some testing on more than one platform (on different hardware, at different screen sizes, on different GNU/Linux distributions) |
| * For new activities: | | * For new activities: |
− | ** Examine the activity for which version of Sugar it supports (focus on 0.82+) | + | ** Examine the activity for which version of Sugar it supports (focus on Sugar 0.82+) |
| * For updates: | | * For updates: |
| ** For activities being updated for 0.84, test on 0.84 as primary and 0.82 as secondary | | ** For activities being updated for 0.84, test on 0.84 as primary and 0.82 as secondary |
Line 80: |
Line 81: |
| | | |
| == Step 4. Security == | | == Step 4. Security == |
| + | |
| + | '''NOTE''': This section needs to be rewritten for the Sugar/Rainbow/Bitfrost context. |
| + | |
| === Does it load remote JavaScript? === | | === Does it load remote JavaScript? === |
| * Deny going public. | | * Deny going public. |
Line 219: |
Line 223: |
| * Support questions that also contain reviews – approve | | * Support questions that also contain reviews – approve |
| | | |
− | [[Category:Activity]] | + | [[Category:Activity Team]] |
| [[Category:HowTo]] | | [[Category:HowTo]] |