search

COSCUP / coscup.org

1 stars 2 forks source link

Simplify Overpass API loader for readibility #23

Closed rileychh closed 1 month ago

rileychh commented 1 month ago

這個 PR 重新整理了 #19 中引入的 Overpass API loader，透過簡化代碼結構讓其更易於理解。

此外，這個 PR 也恢復了 style/no-multi-spaces ESLint 規則的原設定，因為專案已經沒有設定它的需要，而且修改這項設定可能會導致過於寬鬆的排版規則。

雖然這項 PR 不影響 loader 的行為，不過我在本地開發 /event 時會遇到 fetch 逾時 (ETIMEDOUT)，再麻煩大家確認能不能成功載入。 Edit: CI 能編譯，所以應該是我自己的網路問題。

rileychh commented 1 month ago

最好還是確認一下各大瀏覽器都沒這個問題，像是 Chrome, Firefox, Safari, Chrome Android

On Wed, Sep 11, 2024, 23:01 Mirumo @.***> wrote:

@.**** approved this pull request.

LGTM (+ - 的按鈕的底線突然被修正了？！

— Reply to this email directly, view it on GitHub https://github.com/COSCUP/coscup.org/pull/23#pullrequestreview-2297472217, or unsubscribe https://github.com/notifications/unsubscribe-auth/AM3QX66HMWADRXODNE7KLZDZWBLNVAVCNFSM6AAAAABN6W2CLWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEOJXGQ3TEMRRG4 . You are receiving this because you authored the thread.Message ID: @.***>

mirumodapon commented 1 month ago

原本我也是有底線的，但現在突然不見了🤔