bcgov / bcgov/entity

NR UI: delete obsolete "ASSUMED" code

Open
#6,172 0 comments 0 reactions 0 assignees View on GitHub
ENTITY - DO NOT USE Name Request Priority3 techdebt
Dominant language
JavaScript
Stars
23
Forks
62
Avg merge
24m
Merged PRs (30d)
1

Description

Apparently, the ASSUMED state is now obsolete. (It was previously when a user initiated an assumed name flow.)

There remain 18 places in the NR UI code where "assumed" is referenced. This code should be deleted.

(The risk with leaving this code in is that it confuses developers -- what I call the "decoy pattern", ie, something that looks like it's doing something but really isn't.)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.