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.

Sign up for GitHub

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

Jump to bottom

[GH-661] add buyables intermediate table #674

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

[GH-661] add buyables intermediate table #674

wants to merge 25 commits into from

Conversation

Nico-Sanchez
Copy link
Collaborator

@Nico-Sanchez Nico-Sanchez commented Jun 3, 2024 •
edited
Loading

Motivation

Explain why not how.
Closes #661

Summary of changes

Add new Buyables table and its associations

How to test it?

You can explain how you tested it or suggest a way to do so.

Checklist

  • Tested the changes locally.
  • Reviewed the changes on GitHub, line by line.
  • This change requires new documentation.
    • Documentation has been added/updated.

@Nico-Sanchez
Add item costs to item templates config
fda7200
@Nico-Sanchez
Add new ItemCost migration, schema and assocs
4274730
@Nico-Sanchez
Add ItemCost to seeds
89c0721
@Nico-Sanchez
Add currencies behavior for current buy item endpoint
9255d77
@Nico-Sanchez
Remove unnecessary ItemCost table and link ItemTemplate directly with…
cc2b31f 
… CurrencyCost
@Nico-Sanchez
Update items_templates json
a0288d5
@Nico-Sanchez
Update buy_item in ItemController to use new associations
9d8aa02
@Nico-Sanchez
Update @doc tags due to changes in the code
0fb84be
@Nico-Sanchez
Move items templates config to new store config
91462b8
@Nico-Sanchez
Add new store config and move items insertion to CoM Config module
d103c50
@Nico-Sanchez
Add new Store table and its associations, also remove unnecessary pur…
eda95a6 
…chasable? field from item templates
@Nico-Sanchez
Add new StoreController to handle Store requests and Store module for…
e381f57 
… its behavior
@Nico-Sanchez
Change Stores function is_available to is_active because makes more s…
77e2897 
…ense
@Nico-Sanchez
Fix list_items_in_store function return for json encoding
af11799
@Nico-Sanchez
Add item_in_store/3 function to check if item is in given store
9318e2b
@Nico-Sanchez
Update gateway router with store endpoints
9ad570d
@Nico-Sanchez
Update code due to changes in base-branch
e7cb420
@Nico-Sanchez
Update list_items_with_prices/1 due to changes in base branch
a56fa4d
@Nico-Sanchez
Fix buy_item/2 handler in store controller
0b6eb54
@Nico-Sanchez
Delete placeholder endpoint to buy items without a Store
c6e7bd0
@Nico-Sanchez
Add @doc tag for Stores functions
c8a277d
@Nico-Sanchez
Add TODO tag for buy_item endpoint
c8505e8
@Nico-Sanchez
Add :game_id field to Store table
c4960c2
@Nico-Sanchez
Upsert Store instead of inserting only
9e59c75
@Nico-Sanchez
Add new Buyables table and its associations
9a22335
@lotuuu lotuuu added the champions MOBA game label Jun 4, 2024
Base automatically changed from gh-638-add-store-table to main June 7, 2024 16:37
@Nico-Sanchez Nico-Sanchez removed the champions MOBA game label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEATURE] Add buyables table
2 participants
@Nico-Sanchez @lotuuu