|
320 | 320 | <h2 class="govuk-heading-l">Review process</h2>
|
321 | 321 |
|
322 | 322 | <p class="govuk-body">
|
323 |
| - Once you've submitted a contribution, our team will review it against |
324 |
| - these criteria: |
| 323 | + Our review process follows the two-stage approach outlined above, with |
| 324 | + different levels of review depending on the stage of your contribution: |
| 325 | + </p> |
| 326 | + |
| 327 | + <h3 class="govuk-heading-m">Stage 1: Initial proposal review</h3> |
| 328 | + |
| 329 | + <p class="govuk-body"> |
| 330 | + When you first propose a component or pattern, we'll conduct a lightweight |
| 331 | + review to determine if it should move to "In progress" status: |
325 | 332 | </p>
|
326 | 333 |
|
327 | 334 | <ol class="govuk-list govuk-list--number">
|
328 | 335 | <li>
|
329 |
| - <strong>Initial assessment</strong> - We'll check if the contribution meets |
330 |
| - the basic criteria and requirements |
| 336 | + <strong>Proposal assessment</strong> - We'll check if the contribution meets |
| 337 | + the "Useful" and "Unique" criteria |
| 338 | + </li> |
| 339 | + <li> |
| 340 | + <strong>Evidence review</strong> - We'll evaluate the user research and examples |
| 341 | + you've provided |
| 342 | + </li> |
| 343 | + <li> |
| 344 | + <strong>Backlog prioritisation</strong> - If accepted, we'll add it to our |
| 345 | + community backlog and assign priority |
| 346 | + </li> |
| 347 | + <li> |
| 348 | + <strong>Status update</strong> - The component will move from "To be developed" |
| 349 | + to "In progress" status |
331 | 350 | </li>
|
| 351 | + </ol> |
| 352 | + |
| 353 | + <p class="govuk-body"> |
| 354 | + <strong>Timeline:</strong> Initial proposal reviews typically take 1-2 weeks. |
| 355 | + We'll provide feedback on whether your proposal meets the Stage 1 criteria |
| 356 | + and guidance on next steps. |
| 357 | + </p> |
| 358 | + |
| 359 | + <h3 class="govuk-heading-m">Stage 2: Development completion review</h3> |
| 360 | + |
| 361 | + <p class="govuk-body"> |
| 362 | + Once development is complete and before a component can achieve "Baseline |
| 363 | + completed" or "In use" status, we conduct a comprehensive review: |
| 364 | + </p> |
| 365 | + |
| 366 | + <ol class="govuk-list govuk-list--number"> |
332 | 367 | <li>
|
333 | 368 | <strong>Technical review</strong> - Our developers will review the code quality,
|
334 |
| - performance, and technical implementation |
| 369 | + performance, and technical implementation against our coding standards |
335 | 370 | </li>
|
336 | 371 | <li>
|
337 | 372 | <strong>Design review</strong> - Our designers will assess consistency, usability,
|
338 | 373 | and alignment with design principles
|
339 | 374 | </li>
|
340 | 375 | <li>
|
341 | 376 | <strong>Accessibility review</strong> - We'll conduct thorough accessibility
|
342 |
| - testing and evaluation |
| 377 | + testing and evaluation, including assistive technology testing |
| 378 | + </li> |
| 379 | + <li> |
| 380 | + <strong>Usability validation</strong> - We'll review user research evidence |
| 381 | + and may conduct additional research to validate the solution |
| 382 | + </li> |
| 383 | + <li> |
| 384 | + <strong>Versatility assessment</strong> - We'll test the component across |
| 385 | + different contexts, browsers, and devices |
343 | 386 | </li>
|
344 | 387 | <li>
|
345 |
| - <strong>User research validation</strong> - We may conduct additional research |
346 |
| - to validate the solution |
| 388 | + <strong>Documentation review</strong> - We'll ensure all documentation, examples, |
| 389 | + and guidance meet our standards |
347 | 390 | </li>
|
348 | 391 | <li>
|
349 | 392 | <strong>Final decision</strong> - The design system team will make a final
|
350 |
| - decision on inclusion |
| 393 | + decision on moving to "Baseline completed" or "In use" status |
351 | 394 | </li>
|
352 | 395 | </ol>
|
353 | 396 |
|
354 |
| - <h3 class="govuk-heading-m">Timeline expectations</h3> |
355 | 397 | <p class="govuk-body">
|
356 |
| - The review process typically takes 4-6 weeks, depending on the complexity |
357 |
| - of the contribution and our current workload. We'll keep you updated |
358 |
| - throughout the process and provide feedback at each stage. |
| 398 | + <strong>Timeline:</strong> Development completion reviews typically take 4-6 |
| 399 | + weeks, depending on the complexity of the contribution and our current workload. |
| 400 | + We'll keep you updated throughout the process and provide feedback at each |
| 401 | + stage. |
359 | 402 | </p>
|
360 | 403 |
|
361 |
| - <h2 class="govuk-heading-l">What happens if criteria aren't met</h2> |
| 404 | + <h3 class="govuk-heading-m">Our collaborative approach</h3> |
362 | 405 |
|
363 | 406 | <p class="govuk-body">
|
364 |
| - If your contribution doesn't meet all the criteria, we'll work with you |
365 |
| - to: |
| 407 | + Throughout both stages, we provide iterative feedback and support to help |
| 408 | + improve your contribution. We're committed to supporting contributors and |
| 409 | + helping improve contributions rather than simply rejecting them. |
366 | 410 | </p>
|
367 | 411 |
|
368 | 412 | <ul class="govuk-list govuk-list--bullet">
|
369 |
| - <li>Identify specific areas that need improvement</li> |
370 |
| - <li>Provide guidance on how to address any issues</li> |
371 |
| - <li>Suggest alternative approaches if appropriate</li> |
372 |
| - <li>Connect you with relevant expertise within our team</li> |
| 413 | + <li> |
| 414 | + <strong>Regular check-ins</strong> - We'll schedule regular meetings during |
| 415 | + development to provide guidance and feedback |
| 416 | + </li> |
| 417 | + <li> |
| 418 | + <strong>Early feedback</strong> - We encourage sharing work-in-progress to |
| 419 | + get feedback before formal review |
| 420 | + </li> |
| 421 | + <li> |
| 422 | + <strong>Status transparency</strong> - You can track progress through our |
| 423 | + component status system |
| 424 | + </li> |
| 425 | + <li> |
| 426 | + <strong>Identify improvement areas</strong> - If criteria aren't met, we'll |
| 427 | + identify specific areas that need improvement |
| 428 | + </li> |
| 429 | + <li> |
| 430 | + <strong>Practical guidance</strong> - We provide clear guidance on how to |
| 431 | + address any issues |
| 432 | + </li> |
| 433 | + <li> |
| 434 | + <strong>Alternative approaches</strong> - We'll suggest different approaches |
| 435 | + if appropriate |
| 436 | + </li> |
| 437 | + <li> |
| 438 | + <strong>Expert support</strong> - We'll connect you with relevant expertise |
| 439 | + within our team |
| 440 | + </li> |
373 | 441 | </ul>
|
374 | 442 |
|
375 | 443 | <p class="govuk-body">
|
376 |
| - We're committed to supporting contributors and helping improve |
377 |
| - contributions rather than simply rejecting them. Our goal is to build the |
378 |
| - best possible component library together. |
| 444 | + Our goal is to build the best possible component library together, working |
| 445 | + collaboratively to ensure every contribution reaches its full potential. |
379 | 446 | </p>
|
380 | 447 |
|
381 | 448 | <h2 class="govuk-heading-l">Maintaining contributions</h2>
|
|
0 commit comments