Skip to content

Remove Redundant Code (TODOs) #1489

@hfilmore

Description

@hfilmore

What would you like to change?

steps > application > applying-with > template.njk (lines 73-78) & form.njk (lines 50-55) contain commented out code with a comment to be implemented once Save as Draft functionality exists. Functionality does now exist but seems to have been implemented separately and this code appears to no longer be required.

{# TODO uncomment this once we have save as draft fuctionality #}
            {# <p class="govuk-body">
                <button class="hmcts-button-link" type="submit" name="saveandsignout" value="true" data-prevent-double-click="true" data-module="govuk-button"/>
                {{ saveAsDraft }}
                </button>
            </p>#}

There are also another 2 instances of code which may not be in use but requires further investigation:
steps > common > common.content.ts > generatePageContent
const contactEmail = 'todo@test.com';

steps > la-portal > check-your-answers > content.ts > form > fields > todoVar
todoVar: { type: 'hidden', hidden: true },

How do you think that would improve the project?

Cleaner code.

If this entry is related to a bug, please provide the steps to reproduce it

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions