Skip to content
This repository has been archived by the owner on Sep 11, 2021. It is now read-only.

Seoabot Remaster(Rewrite) #69

Open
wants to merge 45 commits into
base: remaster
Choose a base branch
from
Open

Seoabot Remaster(Rewrite) #69

wants to merge 45 commits into from

Conversation

UnderC
Copy link
Member

@UnderC UnderC commented Nov 20, 2019

Remaster 작업은 끝났고 이슈에 있는 기능들을 하나씩 추가할 예정

index.js Outdated Show resolved Hide resolved
* @param {String | Number} perm 디스코드 권한
* @param {Boolean} ownOnly 봇 소유자 전용 여부
*/
constructor (alias, name, commandKey, perm, ownOnly) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

constructor에다 args 여러개 넣는 것 보단 options 같은 Object로 묵거나 상속하는 클래스들에서 다시 this 값들을 선언해주는 것이 좋아요

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

시간이 늦은 관계로 내일부터 수정할 예정입니다

},
{
"language": "py3",
"question": "다음 python3 코드에서 random 값이 3이라면 'Yee'가 출력되는가?",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드가 어디 있나요?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lib/commands/quiz.js에서 사용됩니다.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문에서 다음 python3 코드 라고 하는데 코드가 없는거 같아서요

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

퀴즈 제작하시는분들 일좀 시키죠

package.json Outdated Show resolved Hide resolved
@pmh-only pmh-only changed the base branch from master to remaster March 12, 2020 07:52
dependabot bot and others added 4 commits March 14, 2020 04:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants