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.

Sign up for GitHub

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

Jump to bottom

[Feature] Implement in-application feedback #59

Closed
RyanWalpole opened this issue Jun 12, 2022 · 1 comment
Closed

[Feature] Implement in-application feedback #59

RyanWalpole opened this issue Jun 12, 2022 · 1 comment
Assignees
@RWELabs @RyanWalpole
Labels
type: implementation A feature or bug that will be worked on and implemented.
Projects
Stardew Valley Mod Manager (Stable)
Milestone
v220701_stable

Comments

@RyanWalpole
Copy link
Collaborator

Overview
Users should be able to give feedback. There should be a way from within the app to give feedback, if not, link the user to a place where they can give feedback. This should be fairly high priority.
@RyanWalpole RyanWalpole added type: feature A feature or a request for a feature to be added to the application status: needs triage Requires assessment to determine severity, impact and to collect more data on the issue. labels Jun 12, 2022
@RyanWalpole RyanWalpole added this to the v220601_stable milestone Jun 12, 2022
@RWELabs RWELabs self-assigned this Jun 14, 2022
@RyanWalpole RyanWalpole self-assigned this Jun 14, 2022
RWELabs added a commit that referenced this issue Jun 15, 2022
@RWELabs @RyanWalpole
Update MainPage.cs
94dbc7e 
Implement #59 - feedback link added to main dashboard.
Implement #57 - help link added to main dashboard.

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 15, 2022
@RWELabs @RyanWalpole
Update MainPage.Designer.cs
99d8be8 
Add new help and feedback links, move around link labels to compensate. Related: #59 #57

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
@RyanWalpole RyanWalpole added type: implementation A feature or bug that will be worked on and implemented. and removed type: feature A feature or a request for a feature to be added to the application labels Jun 16, 2022
RWELabs added a commit that referenced this issue Jun 16, 2022
@RWELabs @RyanWalpole
Update MainPage.cs
d0bde82 
- Fix issue #65 in MainPage.cs
- Created dashboard tab for implementation of #59

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 16, 2022
@RWELabs @RyanWalpole
Update MainPage.cs
196d56a 
- Improved feedback tab operations, so that multiple of the tab can't be open at once - #59

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 17, 2022
@RWELabs @RyanWalpole
Feedback UI Update
04a87e1 
Update UI for Feedback tab, adding header, icon and description. CC: #59

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 24, 2022
@RWELabs @RyanWalpole
Update MainPage.cs
e3b635f 
Update #59 feedback implementation

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 24, 2022
@RWELabs @RyanWalpole
Update MainPage.Designer.cs
f20a04f 
Update feedback #59 implementation

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
RWELabs added a commit that referenced this issue Jun 28, 2022
@RWELabs @RyanWalpole
Update MainPage.Designer.cs
e508705 
Update #59 - implementation of feedback to be integrated now with a webbrowser control.

Co-Authored-By: Ryan Walpole <69621127+RyanWalpole@users.noreply.github.com>
@RyanWalpole RyanWalpole mentioned this issue Jun 28, 2022
Merged
@RyanWalpole
Copy link
Collaborator Author

Merge Notes:
I'm still not entirely sold on this implementation of an in-app feedback form. It feels clunky, due to the WebBrowserObject's obsolete rendering engine - that being said, I don't feel like the app warrants the additional install requirement of WebView2 or the extra size of CefSharp for the application just for something so minor. Maybe it is better just to have the link open a webbrowser? Or maybe we should embrace the game tools focus a tad more and provide in-app online tools, e.g. access to wiki, calculators, growth times, etc

@RWELabs RWELabs removed the status: needs triage Requires assessment to determine severity, impact and to collect more data on the issue. label Jul 4, 2022
@RyanWalpole RyanWalpole mentioned this issue Jul 7, 2022
Merged
@RyanWalpole RyanWalpole added this to Fixed In Release in Stardew Valley Mod Manager (Stable) Jul 7, 2022
@RyanWalpole RyanWalpole mentioned this issue Jul 27, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RWELabs RWELabs

@RyanWalpole RyanWalpole

Labels
type: implementation A feature or bug that will be worked on and implemented.
Projects
Stardew Valley Mod Manager (Stable)
Archived in project
Stardew Valley Mod Manager (Stable)
Fixed In Release
Milestone
v220701_stable
Development

No branches or pull requests
2 participants
@RWELabs @RyanWalpole