submit-social-housing-lettings-and-sales-data

2693 Commits

218 Branches

260 Tags

83 MiB

Tree: d6dc0dd739

Commit Graph

Author	SHA1	Message	Date
natdeanlewissoftwire	d4d93ee44c	CLDC-2671 Update duplicate logs routing (#1983 ) * feat: add param to banner duplicate logs route * feat: pass org id param through for support user case * feat: update tests * feat: lint * feat: add one more org id param to change links * feat: add duplicate logs controller tests * feat: update duplicate log tests * feat: tests no duplicates page * feat: test params passed correctly on delete * refactor: lint * refactor: lint	1 year ago
kosiakkatrina	e7684fc2dd	CLDC-2495 Add delete duplicate logs page (#1752 ) * CLDC-2494: WIP * CLDC-2494: wip * CLDC-2494: page work in progress * cleanup * Add a path for duplicate logs * Display all duplicate logs * Move a test * Display duplicate check answers for logs * Add buttons to delete duplicates * Add a route for sales logs duplicates * Update duplicated page to work for sales logs * lint * Add auth * Rebase updates * Update displayed questions * Add delete duplicates page * Update page content for when there are multiple duplicates * Add auth to the path * Update delete button to use delete controller * Update page for sales logs * Update styling * Update success banner for deleting duplicates * Typo * Render not found if there are no duplicate logs * rebase changes * Rebase changes * Update back and cancel links * Update the duplicate journey after deleting logs * Update change button routing * Refactor tests * Add content for no duplicates * Update params * Refactor tests and paths * Update params to include original_log_id from the beginning * Rename file * lint * Add full stop after deletion messages * Add caption and update duplicate log IDs in the banner * lint * Styling --------- Co-authored-by: Aaron Spencer <aaron.spencer@softwire.com>	1 year ago
Arthur Campbell	cec1208b35	CLDC-2290 implement delete multiple logs story (#1657 ) * add a button to the logs list to delete multiple logs style and position of button helpers for displaying the button conditionally depending on user role and what filters and search are active * correct indentation from 4 spaces to 2 in view file * test appearance of delete logs button on index page for lettings logs * write a happy path feature test for the entire journey * create basic tests for the view component for listing logs to delete * create request tests for the GET delete-logs path * create request tests for the GET delete-logs-confirmation path * create request tests for the DELETE delete-logs path * comprehensive reworking after code review ensure that we are not passing lists of ids through params in the query string, risking overflowing the maximum URL length, adjust tests accordingly, do not attempt to reuse the same table for sales and lettings * alter config to allow creating controllers from the command line with associated spec files that matches how we test * extract controller methods and associated tests to do with the delete logs feature into their own controller, amend routes accordingly * implement same work for sales as for lettings * implement the story for lettings and sales logs under the organisation tab routing and controller methods testing for deleting sales logs, lettings or sales logs for an organisation move storage of relevant routes inside the form object as a comprehensive view model * merge the delete pages for lettings logs and sales logs, add to the tests for the lettings page to test sales specific content * minor refactor to delete logs controller: ensure session filters are only fetched from teh session when needed and extract discard logs method to private method * extract tables for lettings and sales to own partials * refactor delete logs controller after tech review improve the private method that builds the form object so that it has the flexibility to do so for all controller methods ensure that the search term is passed to the delete logs controller when navigating through the organisations tab ensure that noly logs for that organisation are displayed when navigating to delete logs through the organisations tab * remove unnecessary untested arguments * test new helper methods * implement dirty fiddle to get the checkboxes smaller and also not misaligned * ensure delete logs button is always visible on log lists when in the organisations tab * minor linting corrections * revert change, causing errors and outside the scope of this ticket * simplify tests for whether delete logs button appears on index page * replicate request specs from lettings for sales and organisations controllers * minor refactor of lettings log feature spec setup, replicate happy path for sales * minor refactors after rebasing onto Nat's work * temp * write tests for the delete logs form object * lint: add new line at end of file * respond to PO feedback the log id in the delte logs table should be a link to the log the delete logs button should be visible when the user is in a bulk upload journey updated associated tests	2 years ago

Author

SHA1

Message

Date

natdeanlewissoftwire

d4d93ee44c

CLDC-2671 Update duplicate logs routing (#1983 )

* feat: add param to banner duplicate logs route

* feat: pass org id param through for support user case

* feat: update tests

* feat: lint

* feat: add one more org id param to change links

* feat: add duplicate logs controller tests

* feat: update duplicate log tests

* feat: tests no duplicates page

* feat: test params passed correctly on delete

* refactor: lint

* refactor: lint

1 year ago

kosiakkatrina

e7684fc2dd

CLDC-2495 Add delete duplicate logs page (#1752 )

* CLDC-2494: WIP

* CLDC-2494: wip

* CLDC-2494: page work in progress

* cleanup

* Add a path for duplicate logs

* Display all duplicate logs

* Move a test

* Display duplicate check answers for logs

* Add buttons to delete duplicates

* Add a route for sales logs duplicates

* Update duplicated page to work for sales logs

* lint

* Add auth

* Rebase updates

* Update displayed questions

* Add delete duplicates page

* Update page content for when there are multiple duplicates

* Add auth to the path

* Update delete button to use delete controller

* Update page for sales logs

* Update styling

* Update success banner for deleting duplicates

* Typo

* Render not found if there are no duplicate logs

* rebase changes

* Rebase changes

* Update back and cancel links

* Update the duplicate journey after deleting logs

* Update change button routing

* Refactor tests

* Add content for no duplicates

* Update params

* Refactor tests and paths

* Update params to include original_log_id from the beginning

* Rename file

* lint

* Add full stop after deletion messages

* Add caption and update duplicate log IDs in the banner

* lint

* Styling

---------

Co-authored-by: Aaron Spencer <aaron.spencer@softwire.com>

1 year ago

Arthur Campbell

cec1208b35

CLDC-2290 implement delete multiple logs story (#1657 )

* add a button to the logs list to delete multiple logs
style and position of button
helpers for displaying the button conditionally depending on user role and what filters and search are active

* correct indentation from 4 spaces to 2 in view file

* test appearance of delete logs button on index page for lettings logs

* write a happy path feature test for the entire journey

* create basic tests for the view component for listing logs to delete

* create request tests for the GET delete-logs path

* create request tests for the GET delete-logs-confirmation path

* create request tests for the DELETE delete-logs path

* comprehensive reworking after code review
ensure that we are not passing lists of ids through params in the query string, risking overflowing the maximum URL length,
adjust tests accordingly,
do not attempt to reuse the same table for sales and lettings

* alter config to allow creating controllers from the command line with associated spec files that matches how we test

* extract controller methods and associated tests to do with the delete logs feature into their own controller,
    amend routes accordingly

* implement same work for sales as for lettings

* implement the story for lettings and sales logs under the organisation tab
routing and controller methods
testing for deleting sales logs, lettings or sales logs for an organisation
move storage of relevant routes inside the form object as a comprehensive view model

* merge the delete pages for lettings logs and sales logs, add to the tests for the lettings page to test sales specific content

* minor refactor to delete logs controller: ensure session filters are only fetched from teh session when needed and extract discard logs method to private method

* extract tables for lettings and sales to own partials

* refactor delete logs controller after tech review
improve the private method that builds the form object so that it has the flexibility to do so for all controller methods
ensure that the search term is passed to the delete logs controller when navigating through the organisations tab
ensure that noly logs for that organisation are displayed when navigating to delete logs through the organisations tab

* remove unnecessary untested arguments

* test new helper methods

* implement dirty fiddle to get the checkboxes smaller and also not misaligned

* ensure delete logs button is always visible on log lists when in the organisations tab

* minor linting corrections

* revert change, causing errors and outside the scope of this ticket

* simplify tests for whether delete logs button appears on index page

* replicate request specs from lettings for sales and organisations controllers

* minor refactor of lettings log feature spec setup, replicate happy path for sales

* minor refactors after rebasing onto Nat's work

* temp

* write tests for the delete logs form object

* lint: add new line at end of file

* respond to PO feedback
the log id in the delte logs table should be a link to the log
the delete logs button should be visible when the user is in a bulk upload journey
updated associated tests

2 years ago

3 Commits (d6dc0dd739c7c6316d9652e2ba36f44d7102b5d1)