Skip to content

Fix - Custom Data key naming - #301

Merged
ItsNature merged 1 commit into
version/1.2.9from
fix/custom-data-key-naming
Aug 10, 2026
Merged

Fix - Custom Data key naming#301
ItsNature merged 1 commit into
version/1.2.9from
fix/custom-data-key-naming

Conversation

@ItsNature

@ItsNature ItsNature commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

Overview

Description:
Renames the item custom data keys documented for the inventory module from camelCase to snake_case, matching Minecraft's resource location conventions. The client accepts both spellings, so existing integrations keep working.

Changes:

  • copyToClipboard -> copy_to_clipboard
  • openUrl -> open_url
  • suggestCommand -> suggest_command
  • runCommand -> run_command
  • hideItemTooltip -> hide_item_tooltip
  • hideSlotHighlight -> hide_slot_highlight

Review Request Checklist

  • Your code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • I have tested this change myself. (If applicable)
  • I have made corresponding changes to the documentation. (If applicable)
  • The branch name follows the projects naming conventions. (e.g. feature/add-module & bugfix/fix-issue)

@ItsNature ItsNature added type: Documentation Documentation improvement or issue type: Enhancement Feature improvement or addition labels Jul 12, 2026
@ItsNature
ItsNature force-pushed the fix/custom-data-key-naming branch from 56fd61d to 9d55144 Compare August 10, 2026 21:04
@ItsNature
ItsNature merged commit e1592bb into version/1.2.9 Aug 10, 2026
2 checks passed
@ItsNature
ItsNature deleted the fix/custom-data-key-naming branch August 10, 2026 21:04
@ItsNature ItsNature mentioned this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: Documentation Documentation improvement or issue type: Enhancement Feature improvement or addition

Development

Successfully merging this pull request may close these issues.

3 participants