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

Add button to toggle Technic machine protection on and off #131

Closed
wants to merge 1 commit into from
Closed

Add button to toggle Technic machine protection on and off #131

wants to merge 1 commit into from

Conversation

Bremaweb
Copy link

No description provided.

@ShadowNinja
Copy link
Member

  1. This doesn't use general functions. (technic.machine_*)
  2. The strings are not translatable.
  3. There are some style issues. (space missing after comma, spaces inside (), extra ())
  4. Please use a capital letter for the first word of commit messages.

@ShadowNinja ShadowNinja changed the title add button to toggle technic machine protection on and off Add button to toggle Technic machine protection on and off Mar 30, 2014
@VanessaE VanessaE closed this in a13e7b7 Feb 6, 2015
@asl97 asl97 mentioned this pull request May 22, 2015
Merged
@asl97
Copy link
Contributor

asl97 commented May 22, 2015

reposting from issue 191 (i didn't link it cause it's already linked above)

a13e7b7 doesn't include support for LV

HybridDog pushed a commit to HybridDog/technic that referenced this pull request Mar 5, 2016
@t4im @HybridDog
add the option for publicly shared machines via an upgrade item "defa…
a770c72 
…ult:chest

the upgrade slots remain protected

this replaces the need for a few forks and also resolves minetest-mods#131 in the process, which is obsolete now
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
Feature Maintenance Needs rebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Bremaweb @ShadowNinja @asl97 @Ekdohibs