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

ContentDB: Add reason to downloads #10876

Merged
merged 3 commits into from Jul 27, 2021
Merged

ContentDB: Add reason to downloads #10876

merged 3 commits into from Jul 27, 2021

Conversation

rubenwardy
Copy link
Member

This PR adds a query parameter to allow ContentDB to distinguish between new installs, dependency installs, and updates.

To do

This PR is Ready for Review.

@SmallJoker SmallJoker added the Rebase needed The PR needs to be rebased by its author. label Feb 27, 2021
@rubenwardy rubenwardy removed the Rebase needed The PR needs to be rebased by its author. label Jun 1, 2021
@SmallJoker
Copy link
Member

Looks good otherwise 👍

builtin/mainmenu/dlg_contentstore.lua Outdated Show resolved Hide resolved
Copy link
Member

@SmallJoker SmallJoker left a comment

Choose a reason for hiding this comment

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

LGTM

@sfan5 sfan5 merged commit 9c145ba into minetest:master Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants