docs(dexit): note NV adapter also mis-parses happy path (not just errors)
This commit is contained in:
parent
76c4d55603
commit
561ad78ea8
1 changed files with 6 additions and 2 deletions
|
|
@ -57,8 +57,12 @@ input selector, and `search_name()` swallows the exception and **defaults to
|
||||||
- This is a **portal-scraping maintenance task**: inspect the current NV (and verify
|
- This is a **portal-scraping maintenance task**: inspect the current NV (and verify
|
||||||
TX) name-search DOM, update `states/nv/adapter.py` (and `tx`) selectors/flow, and
|
TX) name-search DOM, update `states/nv/adapter.py` (and `tx`) selectors/flow, and
|
||||||
make `search_name()` distinguish a real "taken" result from an adapter error
|
make `search_name()` distinguish a real "taken" result from an adapter error
|
||||||
(return an explicit error/unknown state, never a false "taken"). The harness
|
(return an explicit error/unknown state, never a false "taken"). NOTE the adapter
|
||||||
re-run is the acceptance test.
|
also MIS-PARSES on its happy path: via the route a unique name still returns
|
||||||
|
available=false (the search page content does not match the adapter's "no results"
|
||||||
|
phrases), so the fix must update both the input selector AND the results parsing.
|
||||||
|
We did harden the error path: search_name() now returns available=None on a thrown
|
||||||
|
adapter error. The harness re-run is the acceptance test.
|
||||||
|
|
||||||
**Conclusion:** the ERPNext/order plumbing is sound and verified; **do not enable a
|
**Conclusion:** the ERPNext/order plumbing is sound and verified; **do not enable a
|
||||||
self-serve formation checkout until name search is fixed** (a customer must be able
|
self-serve formation checkout until name search is fixed** (a customer must be able
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue