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] QuestTracker v1.0.0.0 #4425

Merged
merged 7 commits into from
Sep 15, 2024
Merged

[Testing] QuestTracker v1.0.0.0 #4425

merged 7 commits into from
Sep 15, 2024

Conversation

isaiahcat
Copy link
Contributor

No description provided.

@bleatbot bleatbot enabled auto-merge (squash) August 18, 2024 20:48
@bleatbot
Copy link
Collaborator

bleatbot commented Aug 18, 2024

Outdated attempt

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

The average merge time for plugin updates is currently 6 hours.

✔️QuestTracker [testing-live] - 35733d9 - v1.0.0.0 - Repo (New plugin)

Show log - Review

@bleatbot bleatbot added new plugin This is a new plugin. size-large Diff for this PR is large. labels Aug 18, 2024
@philpax philpax added pending-code-review This plugin still needs code review. 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 Aug 26, 2024
@philpax
Copy link
Contributor

philpax commented Aug 27, 2024

This plugin is now in the plugin review queue; we hope to get back to you soon!

@philpax philpax added completed-rules-compliance This plugin is compliant with all rules. completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. and removed pending-rules-compliance This plugin still needs to be checked for rules compliance by the majority of the PAC. pending-code-review This plugin still needs code review. labels Aug 29, 2024
@philpax
Copy link
Contributor

philpax commented Sep 1, 2024

bleatbot, rebuild

@bleatbot
Copy link
Collaborator

bleatbot commented Sep 1, 2024

Outdated attempt

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

✔️QuestTracker [testing-live] - 35733d9 - v1.0.0.0 - Repo (New plugin)

Show log - Review

@Critical-Impact
Copy link
Contributor

Test Results:
Plugin does finish loading however I get an error
Error loading QuestData from data.jason
Could not find file 'Z:\home\myname\.xlcore\data.json

When I try to open either of the windows I get

18:37:19.115 | ERR | Exception during raise of "Void DrawUI()"
	System.NullReferenceException: Object reference not set to an instance of an object.
	   at QuestTracker.QuestDataManager.UpdateQuestData(QuestData questData) in /work/repo/QuestTracker/QuestDataManager.cs:line 97
	   at QuestTracker.MainWindow.Draw() in /work/repo/QuestTracker/MainWindow.cs:line 52
	   at Dalamud.Utility.EventHandlerExtensions.HandleInvoke(Action act) in C:\goatsoft\companysecrets\dalamud\\Utility\EventHandlerExtensions.cs:line 124
18:37:20.121 | INF | [LOCALPLUGIN] Finished unloading "QuestTracker"

I'm running XIV on linux, this shouldn't generally make a difference but if you are loading a file of some description, you'll need to make sure your paths are cross platform compatible. If you need help just tag me.

@wolfcomp
Copy link
Contributor

wolfcomp commented Sep 2, 2024

To what Critical posted, I have located the issue and also submitted a PR for fixing it as grabbing files from where the DLL is located is prone to issues PR in question: isaiahcat/QuestTracker#1

@philpax
Copy link
Contributor

philpax commented Sep 3, 2024

Hi @isaiahcat, please merge the above PR, or a similar fix, at your convenience - that's the only thing blocking the merge right now 🙂

@philpax philpax added blocked Blocked on an issue that needs to be resolved before merge completed-testing This plugin has been tested. and removed pending-testing This plugin still needs to be tested. labels Sep 3, 2024
auto-merge was automatically disabled September 4, 2024 17:50

Head branch was pushed to by a user without write access

@isaiahcat
Copy link
Contributor Author

Merged and updated the manifest.toml to point to the latest commit. Thanks for the help, everyone!

@bleatbot
Copy link
Collaborator

bleatbot commented Sep 4, 2024

Outdated attempt

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

❌QuestTracker [testing-live] - f7d56b7 - Build failed (Diff)

Show log - Review

@bleatbot bleatbot added build failed This plugin failed to build. and removed size-large Diff for this PR is large. labels Sep 4, 2024
@Haselnussbomber
Copy link

Haselnussbomber commented Sep 4, 2024

The assembly name was misspelled in the GetManifestResourceStream call (QeustTracker instead of QuestTracker):
isaiahcat/QuestTracker@f7d56b7#diff-8d20cea1ac66c19c4da60f6a04446e7814520326864e634c85dc035299b1217cR33

Also make sure to build for net8.0-windows here: https://github.com/isaiahcat/QuestTracker/blob/master/QuestTracker/QuestTracker.csproj#L10

@philpax
Copy link
Contributor

philpax commented Sep 13, 2024

Hi @isaiahcat, just checking in on this - it looks like you'll have to apply the fixes above and then you should hopefully be good to go 🙏

@isaiahcat
Copy link
Contributor Author

All fixed :) Apologies, I didn't get notified about these comments for some reason

@bleatbot
Copy link
Collaborator

bleatbot commented Sep 13, 2024

Outdated attempt

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

❌QuestTracker [testing-live] - 564ea8e - Build failed (Diff)

Show log - Review

@bleatbot
Copy link
Collaborator

bleatbot commented Sep 13, 2024

Outdated attempt

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

❌QuestTracker [testing-live] - 564ea8e - Build failed (Diff)

Show log - Review

@wolfcomp
Copy link
Contributor

@isaiahcat Forgot to do one thing during the pr, which was including using System.Reflection; which is currently causing the build to fail in QuestDataManager.cs

@bleatbot
Copy link
Collaborator

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

✔️QuestTracker [testing-live] - 2ab4aff - v1.0.0.0 - Repo (New plugin)

Show log - Review

@bleatbot bleatbot added size-large Diff for this PR is large. and removed build failed This plugin failed to build. labels Sep 14, 2024
@philpax philpax merged commit 55adf3c into goatcorp:main Sep 15, 2024
2 checks passed
@philpax
Copy link
Contributor

philpax commented Sep 15, 2024

Congratulations on your first plugin!

As a first-time contributor, if you're in the XIVLauncher & Dalamud Discord server and would like to get access to the private developer channels for plugin feedback and others, please let us know your Discord handle and we'll grant you the role.

@isaiahcat
Copy link
Contributor Author

Thank you!

I'm already in the discord server with my nickname set to isaiahcat :) I already have the "dev channels" role, unless you're talking about something else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked on an issue that needs to be resolved before merge completed-code-review Someone reviewed this. Check if the review wasn't invalidated already. completed-rules-compliance This plugin is compliant with all rules. completed-testing This plugin has been tested. new plugin This is a new plugin. size-large Diff for this PR is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants