takanakahiko / tweet-deleter

Delete your 2 or more days ago tweets Automatically.
MIT License
31 stars 10 forks source link

75asa/add heroku button #16

Closed 75asa closed 3 years ago

75asa commented 3 years ago

What I did

FYI

ref #3

takanakahiko commented 3 years ago

あざます! config が僕向けのものでしかデプロイできないので、そこはいずれどうにかしたほうが良さそうですね

75asa commented 3 years ago

あざます! config が僕向けのものでしかデプロイできないので、そこはいずれどうにかしたほうが良さそうですね

@takanakahiko いきなりPR出しちゃってすみません // そうですね、Redis とか入れて 消したくないやつとか管理した方がいいかもですね〜 環境変数には入れたくない...

takanakahiko commented 3 years ago

今のところ DB には依存していないので、Redis 等で対応するのは避けたいところです。 たとえば以下のように設定を差し替えるようにすると環境変数のみで各々の設定が使えるので程よい感じですかね。

export SETTING_URL="https://github.com/takanakahiko/tweet-deleter/blob/master/src/settings.ts"
75asa commented 3 years ago

@takanakahiko なるほど 確かにその設定方法だと丁度いいですね だいたい利用する際にフォークする前提なら GitHub ID だけの指定でもいいかもですね e.g. 75asa 僭越ながらイシューたてました #17 ざっくりとした内容なので過不足あれば適宜よろしくおねがいします😀