Skip to content

Browser - Offline Queue for a Stock Count: a sample for cs_event-offline_queue - #886

Merged
oblomov-dev merged 3 commits into
mainfrom
claude/inspiring-cray-b1byik
Oct 24, 2026
Merged

oblomov-dev merged 3 commits into
mainfrom
claude/inspiring-cray-b1byik

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Z2UI5_CL_SMP_APP_543 counts stock into a list and lets Save wait in the browser while the backend cannot be reached — kept with what was typed, sent in the order it was made once the network is back (cs_event-offline_queue, experimental, abap2UI5/abap2UI5#2940). Clear List is not queued and fails as usual. The intro strip says how to try it (developer tools → Network → Offline) and that the page itself still needs the network to load.

Also in this PR, because the sample needs a framework that has the constant:

  • A2UI5_PIN → abap2UI5 main (52976a7). The e2e harness builds against it.
  • e2e harness: that framework transpiles with @abaplint/transpiler 2.14, which writes the classes into node/output/project; transpiledDir( ) (scripts/lib/e2e.mjs) finds them there and in node/output on an older pin. Without it the build found 0 of 142 samples.
  • e2e/expected.mjs: Z2UI5_CL_SMP_APP_061. On the 2.14 runtime describe_by_name( \SYST` )answerstype_not_foundon Node, where 2.13 resolved it; on a system the sample works. Recorded with that reason (explicitly *not* a failure on purpose), to be removed when the runtime resolvesSYST` again.
  • Overview catalog, SAMPLES.md, catalogue.json, catalogue-derived.json regenerated; README count 130 → 131.

How to test

  • npm run check: green (abaplint standard + cloud, abap2UI5 linter, catalogues, markers, titles, keywords, prose, rename)
  • npm run downport + abaplint .github/abaplint/abap_702.jsonc: 0 issues
  • npm run e2e (all 142): every sample passes, 061 as recorded, 464 as before
  • 543 in a browser on the e2e backend with the POSTs failing: two saves wait with the toast "Saved offline - n waiting…", no error dialog; on online all three rows are there, in order

Generated by Claude Code

…ine_queue

Z2UI5_CL_SMP_APP_543 counts stock into a list, and lets SAVE wait in the
browser while the backend cannot be reached: kept with what was typed, sent
in the order it was made once the network is back. Clearing the list is not
queued and fails as usual. The intro says how to try it (the browser's
developer tools, Network - Offline) and that the page itself still needs
the network to load.

A2UI5_PIN moves to abap2UI5 main, the first commit with
cs_event-offline_queue, so the e2e harness builds the sample against a
framework that has it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pzBTy4Fu41dBw89fEtCVg
The framework at the new A2UI5_PIN transpiles its objects into
node/output/project (beside the libraries' folders) instead of node/output.
The build found 0 of 142 samples there and failed, and the smoke would have
found no class to start. transpiledDir( ) answers the folder for either
layout, so an older pin keeps working.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pzBTy4Fu41dBw89fEtCVg
…doing

At the new A2UI5_PIN the framework transpiles with @abaplint/runtime and
transpiler 2.14, and describe_by_name( `SYST` ) answers type_not_found on
Node where 2.13 resolved it. On a system the sample works. Recorded with
that reason - not as a failure on purpose - so the smoke stays the signal
for every other sample; the entry goes when the runtime resolves SYST again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pzBTy4Fu41dBw89fEtCVg
@oblomov-dev
oblomov-dev merged commit c27bb1e into main Oct 24, 2026
23 checks passed
@oblomov-dev
oblomov-dev deleted the claude/inspiring-cray-b1byik branch October 24, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants