starred/plugins
1
0
Fork 0
mirror of https://github.com/pelican-dev/plugins.git synced 2026-05-09 08:26:01 +02:00
Code Issues 13 Projects Packages Wiki Activity

[PR #65] [CLOSED] feat: Add egg tag filtering for mclo.gs upload button #93

New issue
Closed
opened 2026-05-07 00:18:03 +02:00 by BreizhHardware · 0 comments
BreizhHardware commented 2026-05-07 00:18:03 +02:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/plugins/pull/65
Author: @zfelyx
Created: 12/28/2025
Status: ❌ Closed

Base: main ← Head: feature/egg-tag-based-log-upload

📝 Commits (3)

  • abe6bcc Add egg tag check to determine log upload availability
  • febc6ad Update README to include tag-based filtering for log upload button
  • 5febd36 Refactor log upload availability check to simplify egg tag validation

📊 Changes

3 files changed (+53 additions, -6 deletions)

View changed files

📝 README.md (+3 -3)
📝 mclogs-uploader/README.md (+37 -1)
📝 mclogs-uploader/src/Filament/Components/Actions/UploadLogsAction.php (+13 -2)

📄 Description

This PR adds the ability to show the mclo.gs upload button only for servers using eggs with the mclogs-updater tag.

Changes

  • Modified UploadLogsAction.php to check for mclogs-updater tag on the server's egg
  • Button is now hidden by default and only shows for eggs with the appropriate tag
  • Added graceful fallback if tag functionality is not available

Usage

  1. Add the mclogs-updater tag to eggs that support log uploading
  2. The upload button will automatically appear only on servers using tagged eggs
  3. Servers with untagged eggs will not show the upload button

Testing

  • Tested with egg that has mclogs-updater tag - button visible ✅
  • Tested with egg without tag - button hidden ✅
  • Tested with offline server - button hidden ✅ #51

Summary by CodeRabbit

  • New Features

    • Added tag-based visibility for MCLogs Uploader — the upload button now appears only for eggs tagged with "mclogs-updater".

  • Documentation

    • Expanded README: installation, configuration (Tag-Based Filtering), usage, requirements, and support section with repo link.
    • Minor install-path text adjustment in main README.

✏️ Tip: You can customize this high-level summary in your review settings.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/plugins/pull/65 **Author:** [@zfelyx](https://github.com/zfelyx) **Created:** 12/28/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `feature/egg-tag-based-log-upload` --- ### 📝 Commits (3) - [`abe6bcc`](https://github.com/pelican-dev/plugins/commit/abe6bccca61f17386ff662328e53b68078870a52) Add egg tag check to determine log upload availability - [`febc6ad`](https://github.com/pelican-dev/plugins/commit/febc6adf9cfb4b9dc4141820ac201a66a65a4057) Update README to include tag-based filtering for log upload button - [`5febd36`](https://github.com/pelican-dev/plugins/commit/5febd36f4c680d96d87c11ff7e51f438ccf8cc0d) Refactor log upload availability check to simplify egg tag validation ### 📊 Changes **3 files changed** (+53 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+3 -3) 📝 `mclogs-uploader/README.md` (+37 -1) 📝 `mclogs-uploader/src/Filament/Components/Actions/UploadLogsAction.php` (+13 -2) </details> ### 📄 Description This PR adds the ability to show the mclo.gs upload button only for servers using eggs with the `mclogs-updater` tag. ## Changes - Modified `UploadLogsAction.php` to check for `mclogs-updater` tag on the server's egg - Button is now hidden by default and only shows for eggs with the appropriate tag - Added graceful fallback if tag functionality is not available ## Usage 1. Add the `mclogs-updater` tag to eggs that support log uploading 2. The upload button will automatically appear only on servers using tagged eggs 3. Servers with untagged eggs will not show the upload button ## Testing - [x] Tested with egg that has `mclogs-updater` tag - button visible ✅ - [x] Tested with egg without tag - button hidden ✅ - [x] Tested with offline server - button hidden ✅ #51 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added tag-based visibility for MCLogs Uploader — the upload button now appears only for eggs tagged with "mclogs-updater". * **Documentation** * Expanded README: installation, configuration (Tag-Based Filtering), usage, requirements, and support section with repo link. * Minor install-path text adjustment in main README. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-07 00:18:03 +02:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
register/email-verification
No results found.
Labels
Clear labels   
pull-request
Mirrored from GitHub Pull Request

No labels
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/plugins#93
No description provided.