On Fri, Aug 13, 2021 at 1:32 AM Raxel Gutierrez <ra...@google.com> wrote: > > This series is a revision to the previous version of the patch series. > The series mainly addresses the review comments from the v2 series [1]. > In doing so, its patches that deal with refactoring and don�t make > user-facing changes are placed at the beginning of this series to make > it easier to merge them as their complexity is low. > > Since v2: > - Patch 1: adds refactoring of <pk> to <patch_id> > - Patch 2: clean up/refactoring of patch detail page > - Patch 3: adds patch-list.js file to modularize script code in template > - Patch 4: change patch meta info toggles from link elements to > buttons > - Patch 5: adds dependency patch that adds the JS Cookie library > to be able to add csrf tokens to requests made client-side using JS > - Patch 6: adds rest.js file to modularize the functionality of making > REST API requests in the client-side > - Patch 7: split from [v2 1/5] to only deal with adding addressed field > to the PatchComment model and changing it�s edit permissions > - Patch 8: adds /comments/<comment_id> endpoint and has its tests now > too. Also, based on review comments: > - fix OpenAPI definition of new `comments/<comment_id> endpoint to > not apply to older versions (< v1.3) of the REST API > - add motivation of adding addressed field to patch comments & reason > behind new endpoint to commit message of patch > - remove `addressed: False` from tests/utils.py creation of patch > comment > - Patch 9: combine styling from [v2 3/5] and functionality from [v2 4/5] > to add addressed status label and buttons with the ability to update > the state manually through the button as before > - Patch 10: remove preclude section and reword features/api sections > > [1] https://lists.ozlabs.org/pipermail/patchwork/2021-July/007001.html > [2] https://lists.ozlabs.org/pipermail/patchwork/2021-July/006944.html
Forgot to mention the [2] footnote is an earlier thread with a mockup that redesigns the patch-detail page. Feel free to leave feedback there :) [snip range-diff] _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork