Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SVR-14] 재료 조회 기능 추가 #42

Merged
merged 1 commit into from
Aug 13, 2023
Merged

Conversation

kth1888
Copy link
Member

@kth1888 kth1888 commented Aug 12, 2023

기존 씬 + 코드 재사용하여 Ingredient 조회 후 파싱 + 스크린에 출력
Subscription을 계속 유지시키므로 HashSet 이용하여 스크린 텍스트가 중복으로 더해져 출력하는 것을 방지
image

@kth1888 kth1888 requested a review from a team as a code owner August 12, 2023 17:25
@kth1888 kth1888 requested review from Atralupus and upa-r-upa and removed request for a team August 12, 2023 17:25
@notion-workspace
Copy link

식재료 조회 (READ)

Copy link
Member

@Atralupus Atralupus left a comment

Choose a reason for hiding this comment

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

MTGL


public class ReadIngredient : MonoBehaviour
{
public string pkHex = "eda6ef63ae945cd15572fcf7d6635a8b3f8d86e85b57a353b482bc82c7fd2ad4";
Copy link
Member

@Atralupus Atralupus Aug 12, 2023

Choose a reason for hiding this comment

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

pk를 static class로 선언해두고 사용하심 중복 제거에 도움 될겁니다


GraphApi.Query readIngredient = SavorReference.GetQueryByName("ReadAllIngredient", GraphApi.Query.Type.Subscription);

readIngredient.SetArgs(new { address = "0x53103C2D7875D2f5f02AeC3075155e268a6e3A94" });
Copy link
Member

Choose a reason for hiding this comment

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

이 address도 동일하게 static으로 선언해두시는게 편할겁니다
더해서 나중에 다른 address로 조회할 수 있도록 변경될 가능성이 상당히 높으니 입력 받도록 미리 해두시는것도...

Copy link
Member Author

Choose a reason for hiding this comment

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

다음 작업때 반영할 수 있도록 해보겠습니다 !!

@kth1888 kth1888 added this pull request to the merge queue Aug 13, 2023
Merged via the queue into main with commit e994513 Aug 13, 2023
1 check passed
@kth1888 kth1888 deleted the feat/read-ingredients branch August 13, 2023 03:46
@kth1888 kth1888 changed the title [SVR-14]feat: Read Ingredients [SVR-14] 재료 조회 기능 추가 Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants