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

Apply changes for Thor planet #333

Merged
merged 11 commits into from
Nov 14, 2024
Merged

Apply changes for Thor planet #333

merged 11 commits into from
Nov 14, 2024

Conversation

U-lis
Copy link
Collaborator

@U-lis U-lis commented Nov 12, 2024

  • Give 5 times more items
  • Give 5 times more mileage
  • No mileage products
  • Images has special name for Thor chain

@U-lis U-lis added the enhancement New feature or request label Nov 12, 2024
@U-lis U-lis added this to the v250.0.0 milestone Nov 12, 2024
@U-lis U-lis requested a review from ipdae November 12, 2024 14:00
@U-lis U-lis self-assigned this Nov 12, 2024
@U-lis U-lis linked an issue Nov 12, 2024 that may be closed by this pull request
@@ -44,6 +44,10 @@ def product_list(agent_addr: str,
category_schema_list = []
purchase_history = get_purchase_history(sess, planet_id, agent_addr)
for category in all_category_list:
# Do not show Mileage category for thor chain
Copy link
Member

Choose a reason for hiding this comment

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

이건 왜 가림처리하는건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

토르 체인에서는 마일리지를 5배 쌓고, 이걸 체인이 종료된 후에 다른 체인으로 이동해주는 기획이 있습니다.
마일리지를 5배 주는 대신 여기에서 마일리지 상품을 못 사게 하자는 기획에 따라 Mileage category 를 가립니다.

for x in product.fav_list])
claim_list = [
{"ticker": x.fungible_item_id,
"amount": x.amount * (5 if receipt.planet_id in (PlanetID.THOR, PlanetID.THOR_INTERNAL) else 1),
Copy link
Member

Choose a reason for hiding this comment

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

토르쪽 보정(*5)가 들어가는 코드가 많은데 아예 모델에 메서드를 만들고 그걸 쓰면 어떨까요.

@U-lis U-lis mentioned this pull request Nov 14, 2024
@U-lis U-lis merged commit d193f53 into development Nov 14, 2024
10 checks passed
@U-lis U-lis deleted the feature/thor branch November 14, 2024 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[IAP] Add thor planet
2 participants