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

🍯⛽️ ↝ Users can now add an item to their inventory from a sector they control #100

Merged
merged 15 commits into from
Feb 29, 2024
Merged

🍯⛽️ ↝ Users can now add an item to their inventory from a sector they control #100

merged 15 commits into from
Feb 29, 2024

Conversation

Gizmotronn
Copy link
Member

We just need to add a discriminator to ensure that multiple rows aren't created for the same item.
Potentially it would also be useful to show which sector they came from.

Gizmotronn and others added 15 commits February 16, 2024 23:27
@Gizmotronn
🚁🧼 ↝ Identifying that no crafting recipes are being inforced on struc…
dc7cce5 
…ture creation for [ GP-11 ] yet
@Gizmotronn
🚁😱 ↝ Merge pull request #98 from Signal-K/GP-11
6f44edc 
🧼🚁 ↝ Identifying that no crafting recipes are being inforced on struc…
@Gizmotronn
👖🍌 ↝ 7 new minerals added, all of them can be part of new sectors [ G…
c8077a2 
…P-10 GP-1 CPW-4 ]. + initialised work on randomising rover images for sector coverUrl value
@Gizmotronn
🧲❓ ↝ [debugging] Method to add mineral deposit resource to inventory …
7c56a32 
…[ GP-10 GP-11 ].
@Gizmotronn
🎩⭐️ ↝ oh that took forever. Can (finally) communicate via post to pap…
d1e79b2 
…yrus.
@Gizmotronn
⚾️⚓️ ↝ working on inventory design for [ GP-10 ] structures
75feedd
@Gizmotronn
🌘🤹🏻‍♂️️ ↝ Structures are now shown more pleasantly
0490e9a
@Gizmotronn
️️🌓🌨️ ↝ Init next-pwa
3680f67
@Gizmotronn
🛟🤾🏻 ↝ Now a PWA
f386bfa
@Gizmotronn
🩶🛴 ↝ Working to integrate initialClassification routes
3eb7431
@Gizmotronn
🐼🇮🇪 ↝ Testing some bug fixes in local env
93567f5
@Gizmotronn
️️🌽🏍️ ↝ Items now created per-sector from API, correct schema, AND re…
eff27bf 
…ndering on per-sector basis for already existing items [ GP-10 GP-11 GP-12 ]
@Gizmotronn
Merge branch 'GP-10' into GP-11
5892401
@Gizmotronn
😿🛟 ↝ Merge pull request #99 from Signal-K/GP-11
0528d71 
🧻😿 ↝ Create a working microservice for adding items
@Gizmotronn
🤾🏻‍♀️🪙 ↝ You can now add the mineral deposit to your inventory
3522dd0
@Gizmotronn Gizmotronn added bug Something isn't working content data labels Feb 29, 2024
@Gizmotronn Gizmotronn self-assigned this Feb 29, 2024
@height Height
Copy link

height bot commented Feb 29, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Gizmotronn
Gizmotronn commented Feb 29, 2024
View reviewed changes
Copy link
Member Author

@Gizmotronn Gizmotronn left a comment

Choose a reason for hiding this comment

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

I've updated inventoryUSERS.55 table>row so that we can see which sector it was added from as a ref to this note

@Gizmotronn Gizmotronn merged commit 81f432a into initialClassification Feb 29, 2024
1 check failed
@Gizmotronn Gizmotronn deleted the GP-10 branch February 29, 2024 17:08
@Gizmotronn Gizmotronn mentioned this pull request Feb 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Gizmotronn Gizmotronn

Labels
bug Something isn't working content data
Projects
Planet Generation
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Gizmotronn