fix: also send testing version in feedback

This commit is contained in:
goat 2022-01-08 21:12:01 +01:00
parent d13ece1bbf
commit 3ee5613517
No known key found for this signature in database
GPG key ID: 7773BB5B43BA52E5
2 changed files with 8 additions and 6 deletions

View file

@ -63,6 +63,7 @@ namespace Dalamud.Interface.Internal.Windows
private string feedbackModalContact = string.Empty;
private bool feedbackModalIncludeException = false;
private PluginManifest? feedbackPlugin = null;
private bool feedbackIsTesting = false;
private int updatePluginCount = 0;
private List<PluginUpdateStatus>? updatedPlugins;
@ -424,7 +425,7 @@ namespace Dalamud.Interface.Internal.Windows
{
if (this.feedbackPlugin != null)
{
Task.Run(async () => await BugBait.SendFeedback(this.feedbackPlugin, this.feedbackModalBody, this.feedbackModalContact, this.feedbackModalIncludeException))
Task.Run(async () => await BugBait.SendFeedback(this.feedbackPlugin, this.feedbackIsTesting, this.feedbackModalBody, this.feedbackModalContact, this.feedbackModalIncludeException))
.ContinueWith(
t =>
{
@ -1237,7 +1238,7 @@ namespace Dalamud.Interface.Internal.Windows
if (!manifest.SourceRepo.IsThirdParty && manifest.AcceptsFeedback)
{
this.DrawSendFeedbackButton(manifest);
this.DrawSendFeedbackButton(manifest, false);
}
ImGuiHelpers.ScaledDummy(5);
@ -1455,7 +1456,7 @@ namespace Dalamud.Interface.Internal.Windows
if (canFeedback)
{
this.DrawSendFeedbackButton(plugin.Manifest);
this.DrawSendFeedbackButton(plugin.Manifest, plugin.IsTesting);
}
if (availablePluginUpdate != default)
@ -1688,13 +1689,14 @@ namespace Dalamud.Interface.Internal.Windows
}
}
private void DrawSendFeedbackButton(PluginManifest manifest)
private void DrawSendFeedbackButton(PluginManifest manifest, bool isTesting)
{
ImGui.SameLine();
if (ImGuiComponents.IconButton(FontAwesomeIcon.Comment))
{
this.feedbackPlugin = manifest;
this.feedbackModalOnNextFrame = true;
this.feedbackIsTesting = isTesting;
}
if (ImGui.IsItemHovered())

View file

@ -23,7 +23,7 @@ namespace Dalamud.Support
/// <param name="reporter">The reporter name.</param>
/// <param name="includeException">Whether or not the most recent exception to occur should be included in the report.</param>
/// <returns>A <see cref="Task"/> representing the asynchronous operation.</returns>
public static async Task SendFeedback(PluginManifest plugin, string content, string reporter, bool includeException)
public static async Task SendFeedback(PluginManifest plugin, bool isTesting, string content, string reporter, bool includeException)
{
if (content.IsNullOrWhitespace())
return;
@ -33,7 +33,7 @@ namespace Dalamud.Support
Content = content,
Reporter = reporter,
Name = plugin.InternalName,
Version = plugin.AssemblyVersion.ToString(),
Version = isTesting ? plugin.TestingAssemblyVersion?.ToString() : plugin.AssemblyVersion.ToString(),
DalamudHash = Util.GetGitHash(),
};