Skip to content

fix(bank): 定期存款可以支取 - #247

Merged
Carinoasd merged 2 commits into
masterfrom
fix/bank-fixed-deposit
Oct 26, 2026
Merged

Carinoasd merged 2 commits into
masterfrom
fix/bank-fixed-deposit

Conversation

@Carinoasd

@Carinoasd Carinoasd commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

問題

社區銀行的定期列表裡,每筆記錄只有一個「支取」連結(op=out&noid=…),點進論壇的確認頁才有支取表單;App 只在列表頁找表單,所以找不到支取入口。

修改

  • 定期記錄的支取連結(同站、同銀行、action=fix&op=out、數字 noid,不多帶參數)變成該筆的「支取」按鈕,顯示該筆金額/利率/到期時間。
  • 送出時先重新讀列表核對,再讀該筆的確認頁;確認頁的表單要是同一筆(bankid、op、noid 相同,只要銀行密碼)才送出一次 POST,帶上確認頁自己的 formhash(若有)。
  • 點「支取」時先讀該筆的確認頁:確認頁自己的說明跟該筆記錄一起顯示在對話框;確認頁改顯示原因(例如未到期、已支取)時,只顯示原因、不出現密碼欄、不送出。
  • 送出時再讀一次確認頁,說明文字和剛才顯示的不同就不送出。
  • 其他帶 op 的連結(貸款的還款/取消是點連結就執行)標成 App 不支援,引導到網頁,不在 App 裡執行。

測試

  • 新增 test_264_bank_term_withdraw_test.dart(依 bank_ane 模板結構的合成頁)。
  • 銀行相關測試 test_120–125、264 全過;未在正式站實際支取。

🤖 Generated with Claude Code

Carinoasd and others added 2 commits October 26, 2026 10:16
The term list links each record to a confirmation page (op=out&noid=…) that
holds the withdrawal form, so the app found nothing to withdraw with. A record's
link now becomes its withdrawal: sending reads that page, checks its form is the
one for this record (bank, op, noid, password only) and posts it once; a reason
the page shows instead is displayed and nothing is sent. Other operation links
(a loan's repayment runs on the link itself) are marked unsupported and left to
the website.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…assword

Tapping a term record's withdrawal now reads the forum's confirmation page
first: its own notice is shown with the record in the dialog, and a reason it
gives instead (not due, already withdrawn) is shown without a password field.
The notice shown is checked again when sending; a page saying otherwise by
then sends nothing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Carinoasd
Carinoasd merged commit bb6b34e into master Oct 26, 2026
6 checks passed
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.

1 participant