스팀코인판에 코드기여하기 #3 PR 승인 그리고 되돌리기

sct.jcob (52) in sct • 5 years ago (edited)

스콧 토큰 보기

현재는 잠깐 적용됐다 취소됐습니다.

@realmankwon님이 적용하신 스콧 토큰 보기입니다!

해프닝, 사고

이번에 해프닝이 하나 발생했었는데요.

@realmankwon이 올려주신 PR을 제가 로컬에서 테스트 했고,

잘 동작한다고 판단하여 PR을 승인했습니다.

근데 토큰이 적은 계정에서는 문제가 없었는데

토큰이 많은 계정에서는 에러가 발생해서 다른페이지로 이동이 안됐습니다.

그래서 급하게 PR한 것을 취소하게 됐습니다.

pr을 취소하기 전까지, 30분 정도 지갑이 동작안했는데요, 유저님들에게 죄송합니다.

앞으로 코드 승인은 좀더 철저히 해도록 하겠습니다.

소감

오늘은 머지와 revert 되돌리기를 해봤습니다.

깃헙을 통해 원격에서 협업, 이런 작업을 해본적이 없었습니다. 재밌기도 하고요 신기하기도 하고요.

조금 피곤해도 기여를 해주시는 개발자님들에게 너무 감사하고 고맙습니다!

스크린샷, 2019-06-20 21-47-53.png

스크린샷, 2019-06-20 21-48-33.png

지난 글입니다.

감사합니다!

sct kr-dev suggest kr contribute

5 years ago by sct.jcob (52)

$0.15

Authors get paid when people like you upvote their post.
If you enjoyed what you read here, create your account today and start earning FREE STEEM!

Sort Order:

Trending

[-]

steemitboard (66) · 5 years ago

Congratulations @sct.jcob! You have completed the following achievement on the Steem blockchain and have been rewarded with new badge(s) :

You got your First payout

_{You can view your badges on your Steem Board and compare to others on the Steem Ranking}
_{If you no longer want to receive notifications, reply to this comment with the word STOP}

Do not miss the last post from @steemitboard:

	The Steem community has lost an epic member! Farewell @woflhart!
	SteemitBoard - Witness Update
	Do not miss the coming Rocky Mountain Steem Meetup and get a new community badge!

Vote for @Steemitboard as a witness to get one more award and increased upvotes!

$0.00

[-]

realmankwon (83) · 5 years ago

저도 이래저래 소스를 수정하고 merge하면서 시행착오를 많이 겪고 있네요 ^^
어제 올렸던 소스가 충돌이 난 것 같아서 해당 건은 closed 시키고 다시 pull 받아서 했던 제 로컬의 소스가 다 날라가는 불상사가 발생하였고 이래저래 수정하여 올렸는데 또 소스가 동기화가 안 되서 2번 정도를 더 closed 하고 올렸습니다.
이것도 다 배워가는 과정이라 생각하고 있습니다. ^^

일단 어제 변경한 steem engine token 외에 voting list 와 voting rate 를 별도의 branch로 생성하여 pr 을 올렸습니다.
추후 이 버전 2개가 이상없다고 판단되어 merge가 되면 제 branch는 삭제하고 pull 받은 후 작업을 하도록 하겠습니다.

그리고 스팀엔진측에서 제시한 voting price는 아무래도 이상이 있어 보입니다.
주고 받은 메세지 중 제일 마지막 버전으로 해도 다른 값이 나오고 있습니다.
특히나 현재 voting list 를 보여주는 소스 자체가 각 voter 별 rshares의 크기로만 비교하여 정렬을 하고 있습니다.

       const avotes = active_votes.toJS();
        avotes.sort(
            (a, b) =>
                Math.abs(parseInt(a.rshares)) >
                Math.abs(parseInt(b.rshares))
                    ? -1
                    : 1
        );

바쁘시겠지만 관련하여 확인만 되면 기존의 방식대로 적용이 가능할 듯 합니다.

감사합니다 좋은 밤 되세요 ^^

$0.00

[-]

jacobyu (76) · 5 years ago

추후 이 버전 2개가 이상없다고 판단되어 merge가 되면 제 branch는 삭제하고 pull 받은 후 작업을 하도록 하겠습니다.

안녕하세요. 브랜치를 삭제하지마시고, 리얼맨님 브랜치에서 최신 scotty_steemcoinpan을 머지한 후,
conflict해결하고 잘동작되는 것 확인 한 다음에

그걸 pr 날리면 되지 않을까요?

저도 하나 기여할걸 찾아서 pr날려봐야겠어요 ㅎㅎ

감사합니다!

$0.00