guiconstants: update ORG_DOMAIN to bitcoincore.org #818

1. 
   MarnixCroes commented at 7:00 pm on April 24, 2024: contributor

   if desired, can also update ORG_NAME to Bitcoin Core

2. qt/guiconstants: update ORG_DOMAIN to bitcoincore.org 1b7abd2e4e
3. 
   DrahtBot commented at 7:00 pm on April 24, 2024: contributor

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Reviews

   See the guideline for information on the review process. A summary of reviews will appear here.

4. 
   laanwj commented at 7:31 pm on April 24, 2024: member

   if desired, can also update ORG_NAME to Bitcoin Core

   The reason that this was never done is because it changes where the Qt settings files are stored. For example this is ~/.config/<ORG_NAME>/<APP_NAME> on Linux.

   i’m not sure what actually is the effect of changing the domain, but this shouldn’t be done without checking what it is used for.

5. 
   hebasto commented at 7:49 pm on April 24, 2024: member

   if desired, can also update ORG_NAME to Bitcoin Core

   Are there any issues with the current code? If not, then this PR lacks motivation.

6. 
   MarnixCroes commented at 0:04 am on April 25, 2024: contributor

   ok, it came across in the qt-qml repo, I looked here and assumed it was forgotten to be updated. nvm
7. MarnixCroes closed this on Apr 25, 2024

   ---

8. MarnixCroes deleted the branch on Apr 25, 2024
9. 
   laanwj commented at 8:15 am on April 25, 2024: member

   Right. i think qt-qml does something substantially different with regard to settings, so it may be warranted there.

Contributors
MarnixCroes DrahtBot laanwj hebasto