Delete Feedback section #7

Closed
opened 3 years ago by jahway603 · 10 comments
Collaborator

I was not able to determine how to fix this, so posting this as an issue.

How to reproduce

  1. When you click on Help, the first item states: "Send DenioD Feedback".
  2. The first item there should have Duke's name.
  3. The first item there should mimic the behavior of SD for this functionality.
I was not able to determine how to fix this, so posting this as an issue. How to reproduce 1. When you click on Help, the first item states: "Send DenioD Feedback". 1. The first item there should have Duke's name. 1. The first item there should mimic the behavior of SD for this functionality.
duke was assigned by jahway603 3 years ago
Owner

@jahway603 better yet, just delete it

@jahway603 better yet, just delete it
duke changed title from Change "DenioD" to "Duke" in the Help section to Delete Feedback section 3 years ago
Owner

@jahway603 what about "contact the hush developers" and we use a zaddr which is shared among Hush devs, so that many people can potentially see the feedback? Otherwise it probably gets lost in a wallet.dat somewhere

@jahway603 what about "contact the hush developers" and we use a zaddr which is shared among Hush devs, so that many people can potentially see the feedback? Otherwise it probably gets lost in a wallet.dat somewhere
Owner

@onryo thoughts on this?

@onryo thoughts on this?
Poster
Collaborator

@duke If we're not using the "Send XYZ Feedback" functionality, then it could be removed from code.

@duke If we're not using the "Send XYZ Feedback" functionality, then it could be removed from code.
Poster
Collaborator

I just seen you mentioned an issue like this in Telegram and you sez it's on the Gitea, but I can't find it anywhere @onryo . Anyhow, might have a duplicate issue.

I just seen you mentioned an issue like this in Telegram and you sez it's on the Gitea, but I can't find it anywhere @onryo . Anyhow, might have a duplicate issue.
Collaborator

I could fix this if you want to keep it or remove it entirely. Currently, the action is commented out here and won't do anything when clicking:
https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/mainwindow.cpp#L112

The donate function that would call if not commented out is here:
https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/mainwindow.cpp#L899

getDonationAddr is defined here and currently set to: zs1fq9f7vg797qaeac9lyx0njyjmjg4w7m60hwq6lhyhvdcqltl5hdkm8vwx9cxy60ehuuz2x49jxt
https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/settings.cpp#L365

I could fix this if you want to keep it or remove it entirely. Currently, the action is commented out here and won't do anything when clicking: https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/mainwindow.cpp#L112 The donate function that would call if not commented out is here: https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/mainwindow.cpp#L899 `getDonationAddr` is defined here and currently set to: `zs1fq9f7vg797qaeac9lyx0njyjmjg4w7m60hwq6lhyhvdcqltl5hdkm8vwx9cxy60ehuuz2x49jxt` https://git.hush.is/hush/SilentDragonLite/src/branch/master/src/settings.cpp#L365
Poster
Collaborator

@fekt I think we should remove it if we're not using it.
@duke Do we want to use this instead of deleting it?

@fekt I think we should remove it if we're not using it. @duke Do we want to use this instead of deleting it?
Owner

@fekt @jahway603 let's just delete all the donation related stuff, no idea who even owned that zaddr. If somebody wants to volunteer to get feedback via zaddr, I am fine with that, otherwise just delete it.

@fekt @jahway603 let's just delete all the donation related stuff, no idea who even owned that zaddr. If somebody wants to volunteer to get feedback via zaddr, I am fine with that, otherwise just delete it.
Collaborator

It's removed and comitted: 4efcbc630e

Editor decided to fix some indentation below it I guess.

It's removed and comitted: https://git.hush.is/hush/SilentDragonLite/commit/4efcbc630e9e900236648aa631d01f86d06088ed Editor decided to fix some indentation below it I guess.
Owner

@fekt thanks!

@fekt thanks!
duke closed this issue 2 years ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.