-
-
Notifications
You must be signed in to change notification settings - Fork 1
Add comprehensive ServiceIO PlaceholderAPI documentation #98
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
Conversation
|
@NonSwag 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review. |
Co-authored-by: NonSwag <54660361+NonSwag@users.noreply.github.com>
Co-authored-by: NonSwag <54660361+NonSwag@users.noreply.github.com>
|
You now have the appropriate permissions to perform tests and access the ServiceIO repository |
…ource code Co-authored-by: NonSwag <54660361+NonSwag@users.noreply.github.com>
I've analyzed the ServiceIO source code and updated the documentation with the complete list of all available placeholders. The analysis revealed many more placeholders than initially documented: Key findings:
The documentation now includes 20+ placeholders with practical examples for different use cases including world-specific economy, multi-bank setups, and advanced features. Commit: fb36155 |
| 4. ServiceIO has successfully detected and registered the service provider | ||
| </Callout> | ||
|
|
||
| If a placeholder returns empty or default values, check your ServiceIO configuration and ensure the appropriate service providers are properly installed and configured. No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there will never be "default values"
there is only improper placeholder usage (not formatted and shows the literal placeholder)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated troubleshooting text to clarify that placeholders show literal text instead of expected values when improperly used, rather than mentioning "default values" in commit 6db60d1
|
@copilot |
Co-authored-by: NonSwag <54660361+NonSwag@users.noreply.github.com>
I've addressed all the requested changes in commit 6db60d1:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…ked documentation link
… and PlaceholderAPI link
…callout description
This PR adds comprehensive documentation for ServiceIO's PlaceholderAPI integration introduced in version 2.3.0, based on direct analysis of the ServiceIO source code.
What's Added
New Documentation Page:
content/docs/serviceio/placeholders.mdxDocuments all ServiceIO placeholders using the format
%serviceio_<placeholder>%, including:prefix,suffix,displaynamebalance,balance_<world>,balance_formatted,balance_formatted_<world>bank,bank_balance,bank_balance_formattedwith world-specific variantsbanks,banks_<world>for listing available banksgroup,groups%vaultunlocked_*%placeholders for compatibility reasonsKey Features
Navigation Update: Added the new placeholders page to the ServiceIO documentation section in
meta.json, positioned logically in the Introduction section.The documentation now accurately reflects all 20+ placeholders available in the ServiceIO codebase and provides developers with everything needed to implement ServiceIO placeholders in their configurations.
Fixes #97.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.