Update/burrito to goose #10

ekgns33 · 2023-10-24T03:48:45Z

프로젝트 코드내에 Burrito를 Goose로 변경

작업내역

src 디렉토리 내 burrito를 goose로 변경 (복수형은 geese로 변경)
기존의 BurritoStorage를 GooseStorage로 변경
test 디렉토리 내 burrito를 goose로 변경 (/test/data/* 는 변경x)

특이사항

웹 페이지에 보여지는 부리또 이미지와 css관련된 것은 수정하지 않았습니다.
src/lib/util에
```
const defaultTheme: string = 'https://github.com/chralp/heyburrito-theme';
```
해당 라인은 Theme이 어떻게 될지 몰라 수정하지 않았습니다.

l2hyunwoo

리뷰확인좀

l2hyunwoo · 2023-10-24T04:15:57Z

src/config/index.ts

-      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':burrito:'),
-      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottenburrito:'),
+      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':goose:'),
+      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottengoose:'),


rottenGoose 없애고 여기는 아예 undefined로 정의합시다.

아하 undefined로 정의한다고 하면 env값에는 아무것도 설정하지 않고, ‘rottengoose’를 undefined로 바꾼다는 말씀이신가요???

env값에는 아무것도 설정하지 않고
yes

‘rottengoose’를 undefined로 바꾼다는 말씀이신가요???

이런식으로

Suggested change

emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottengoose:'),

emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC),

l2hyunwoo · 2023-10-24T04:16:04Z

src/config/index.ts

-      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':burrito:'),
-      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottenburrito:'),
+      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':goose:'),
+      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottengoose:'),


l2hyunwoo · 2023-10-24T04:16:08Z

src/config/index.ts

-      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':burrito:'),
-      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottenburrito:'),
+      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':goose:'),
+      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottengoose:'),


l2hyunwoo · 2023-10-24T04:16:55Z

src/config/index.ts

      api_token: mustHave('SLACK_API_TOKEN'),
-      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':burrito:'),
-      emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC || ':rottenburrito:'),
+      emojiInc: fixEmoji(process.env.SLACK_EMOJI_INC || ':goose:'),


할로윈이니까 🍬 (: candy :) 추가하는 것도 괜찮을듯(기간제)
+) 🪿 없을거야, 🦆(: duck :)으로 갑시다

우선 duck으로 적용시키겠습니다. 이모지자체만 (: duck :)으로 바꾸고 변수명등 나머지는 그대로 goose로 유지하나요?

좀 옆길로 새본다면 슬랙 커스텀 이모지로 :goose: 만들수 있긴합니다. 이미지만 있다면•••

@hughesgoon 하긴 그렇긴 함ㅋㅋㅋ 그러면 🪿 일단 추가하고 건구스 사진 추가하고 🪿이모지 만드는 방향으로 가죠! @ekgns33

@l2hyunwoo 앗 duck으로 적용하고 코멘트가 생겼네요 ㅜㅜ default를 duck으로 두고 env에서 goose를 적용시키는 방법은 별로일까요?

@ekgns33 그 방향으로 가시죠!

l2hyunwoo · 2023-10-24T07:03:40Z

Test 깨졌는데 한번 확인해주시겠어요?

l2hyunwoo

gogo!

ekgns33 added 4 commits October 24, 2023 12:27

update: config파일 부리또에서 구스로 변경

e9e6045

update: BurritoStore 을 GooseStore로 교체

f4cf62f

update: src 디렉토리 내 burrito를 goose로 변경, GooseStore 적용

5d8ca77

update: test 디렉토리 내 burrito를 goose로 변경

29eabce

ekgns33 requested a review from a team as a code owner October 24, 2023 03:48

l2hyunwoo suggested changes Oct 24, 2023

View reviewed changes

ekgns33 added 4 commits October 24, 2023 15:07

update : rottengoose 삭제 및 일일 최대 한도 7로 default설정

507ab20

update : 사용자 간 주고받는 이모지 default값 🦆으로 변경

a7592d5

update: 슬랙봇이 사용자에게 goose사용한도 초과시 보내는 메세지의 이모지 수정

46b7ecc

fix : production환경에서 decrease 비활성화

a161619

fix : fixEmoji function에서 undefined를 입력으로 받았을 때 생기는 에러처리

6005396

l2hyunwoo approved these changes Oct 24, 2023

View reviewed changes

ekgns33 merged commit 93f0235 into develop Oct 24, 2023
2 checks passed

ekgns33 deleted the update/burrito-to-goose branch October 24, 2023 07:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update/burrito to goose #10

Update/burrito to goose #10

ekgns33 commented Oct 24, 2023

l2hyunwoo left a comment

l2hyunwoo Oct 24, 2023

ekgns33 Oct 24, 2023

l2hyunwoo Oct 24, 2023

l2hyunwoo Oct 24, 2023

l2hyunwoo Oct 24, 2023

l2hyunwoo Oct 24, 2023

ekgns33 Oct 24, 2023

hughesgoon Oct 24, 2023

l2hyunwoo Oct 24, 2023

ekgns33 Oct 24, 2023

l2hyunwoo Oct 24, 2023

l2hyunwoo commented Oct 24, 2023

l2hyunwoo left a comment

	emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC \|\| ':rottengoose:'),
	emojiDec: fixEmoji(process.env.SLACK_EMOJI_DEC),

Update/burrito to goose #10

Update/burrito to goose #10

Conversation

ekgns33 commented Oct 24, 2023

프로젝트 코드내에 Burrito를 Goose로 변경

작업내역

특이사항

l2hyunwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

l2hyunwoo commented Oct 24, 2023

l2hyunwoo left a comment

Choose a reason for hiding this comment