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

[Testing] MogMogCheck v3.0.1 #5458

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Haselnussbomber
Copy link

@Haselnussbomber Haselnussbomber commented Feb 20, 2025

MogMogCheck is a small plugin to assist you in tracking the rewards of the Moogle Treasure Trove event (called Mog Mog Kollektion in German, hence the name).

Please note that the plugin reads the games shop data, which does not yet contain rewards for the Hunt for Phantasmagoria. Until Patch 7.18 is released, it will show rewards for the Hunt for Goetia.

@bleatbot bleatbot enabled auto-merge (squash) February 20, 2025 11:57
@Haselnussbomber
Copy link
Author

Haselnussbomber commented Feb 20, 2025

Heya, first time submitting here. (Totally not shaking right now. peepoShy)

My plan is to move the plugin to stable next week when the Moogle Tresure Trove starts.
Hope I did everything right with the submission and that it works seamlessly with moving over from a custom repo. 🙂

Regarding the plugin itself, I originally wrote it 2 years ago. It's a seasonal plugin, which means I only update it when a new Moogle Treasure Trove is about to start and a Dalamud API increase makes it mandatory to update it.

Version 3.0.0 was a complete rewrite. I switched over to a modified Dalamud Table class, used a bunch new generators for dependency injection and generally had to update a bunch since my library changed a lot in the time the plugin last got an update.

The plugin supports all game languages where possible (that means data from sheets, like item names and descriptions) and contains translations for the table headers, configuration window, command handler help text and context menus in English and German. I also made sure it works well with any UI scale and that everything is functional.

So, I don't expect any feedback on my code. You can do that of course, but it'd be fine for me if you just look for any malicious stuff that I don't have.^^

Maybe I'll submit my other plugins in the futute!

auto-merge was automatically disabled February 20, 2025 14:52

Head branch was pushed to by a user without write access

@bleatbot
Copy link
Collaborator

bleatbot commented Feb 20, 2025

Outdated attempt

This is the first time that you have submitted a plugin here. Before the bot will build your plugin within the 'Build PR' check, someone from the approval team will need to enable builds for you.

Once this is enabled, the bot will automatically build the PR. Future iterations will not require an approval for building the PR, only merging.

Please hold!

@Haselnussbomber Haselnussbomber changed the title [Testing] MogMogCheck v3.0.0 [Testing] MogMogCheck v3.0.1 Feb 20, 2025
@reiichi001
Copy link
Contributor

bleatbot, approve

@bleatbot
Copy link
Collaborator

bleatbot commented Feb 20, 2025

Outdated attempt

All builds OK!

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

Name Commit Status
✔️ MogMogCheck [testing-live] 4475250 v3.0.1.0 - Repo (New plugin)
13 Needs (⚠️ 5 UNREVIEWED)
Type Name Version Reviewed by
Submodule https://github.com/Haselnussbomber/HaselCommon.git 85982f6bc48b8a8a4165ebdd40bab8fe7b40e66f ⚠️ NEW
NuGet AutoCtor 2.5.0 ⚠️ NEW
NuGet Injectio 4.1.0 ⚠️ NEW
NuGet Haselnussbomber.YogaSharp 1.1.3 ⚠️ NEW
NuGet System.Diagnostics.DiagnosticSource 9.0.2 ⚠️ Upd. from 7.0.2
8 hidden needs (known safe NuGet packages).
Show log - Review

@bleatbot bleatbot added new plugin This is a new plugin. size-large Diff for this PR is large. labels Feb 20, 2025
@Haselnussbomber

This comment was marked as resolved.

@bleatbot
Copy link
Collaborator

All builds OK!

Take care! Please test your plugins in-game before submitting them here to prevent crashes and instability. We really appreciate it!

Name Commit Status
✔️ MogMogCheck [testing-live] ce1f340 v3.0.1.0 - Repo (New plugin)
12 Needs (⚠️ 4 UNREVIEWED)
Type Name Version Reviewed by
Submodule https://github.com/Haselnussbomber/HaselCommon.git 85982f6bc48b8a8a4165ebdd40bab8fe7b40e66f ⚠️ NEW
NuGet AutoCtor 2.5.0 ⚠️ NEW
NuGet Injectio 4.1.0 ⚠️ NEW
NuGet System.Diagnostics.DiagnosticSource 9.0.2 ⚠️ Upd. from 7.0.2
8 hidden needs (known safe NuGet packages).
Show log - Review

@reiichi001 reiichi001 added completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. labels Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. new plugin This is a new plugin. pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-testing This plugin still needs to be tested. size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants