Skip to content

fix: 銀行支取與來源標記的功能審查修正 - #248

Merged
Carinoasd merged 1 commit into
masterfrom
fix/review-1491
Oct 26, 2026
Merged

Carinoasd merged 1 commit into
masterfrom
fix/review-1491

Conversation

@Carinoasd

Copy link
Copy Markdown
Owner

來源

1.49.0/1.49.1 新功能的獨立功能審查(銀行定期支取、「来自天使动漫App」)。銀行部分對照了 bank_ane 的正式站模板、語言字串與伺服器程式。

銀行

  • 讀確認頁失敗(中):原本網路失敗也顯示「此帳戶或頁面暫時無法在 App 內交易」;現在網路失敗提示重試,只有頁面 App 無法處理時才指向網頁。讀取時按鈕顯示轉圈。
  • 貸款頁(低):還貸/取消申請在論壇是點連結就直接執行,改成專屬提示「請到論壇網頁辦理」,不再顯示一般的「無法交易」;營業大廳公告裡(管理員的 BBCode)帶 op 的連結不再讓頁面變成不支援。
  • 送出前頁面變了(低):例如定期剛好到期利息變動、確認頁說明不同,原本整頁變成失敗並清空;現在顯示最新內容並提示「這次沒有送出,請核對後再操作」。仍然不會送出。
  • 論壇回覆(低,舊問題):送出後顯示論壇的回覆文字(例如「银行密码错误,请返回修改。」),結果仍標示未確認。
  • 翻譯(低):支取按鈕與對話框標題改用三語字串「支取定期」。

來源標記

  • 網頁解析只在樓層標頭(div.pti)找插件的標記;帖子內文裡同樣的 HTML(論壇允許 HTML 時)不再被當成來自 App。

測試

  • test_264 改用照正式站模板與語言字串的頁面,補:論壇回覆、密碼錯誤、記錄已不存在、利息變動、確認頁不同、讀取轉圈、讀取失敗與不支援的分別、三語標題。
  • test_263 補:真實樓層標頭的標記、內文裡的偽造標記。
  • 銀行相關與受影響測試:112 項+174 項全過;flutter analyze 無新問題。
  • 未在正式站實際送出。

🤖 Generated with Claude Code

Bank (review of #247, checked against the bank plugin's own templates,
language strings and server code):
- Reading a record's confirmation page: a failed read says so and offers a
  retry; only a page the app cannot take points to the website. The button
  shows the wait.
- A loan's repayment and cancellation run on the link itself: they get their
  own "do it on the website" note instead of the generic unsupported one, and
  links elsewhere (a bank's BBCode notice) no longer mark a page unsupported.
- The page changing before sending (a deposit's interest once due, another
  confirmation notice) shows the page as it is now with a note, instead of a
  failed, empty page. Nothing is sent, as before.
- The forum's reply to a transaction (a wrong bank password, success) is
  shown; the result stays unconfirmed.
- The withdrawal's button and dialog are translated.

Source label: the plugin's span is only taken from the floor's header, not
from a message that holds the same markup (a forum allowing HTML).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Carinoasd
Carinoasd merged commit f70483f 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