• @malloc
    link
    English
    8
    edit-2
    7 months ago

    Whoever committed this change thinks updating the text on the UI for this system will automagically fix issue.

    It’s like editing the HTML on your bank’s website to show you have $1B in the bank 😂

    Issue here committer forgot to update UI to expose the feature person was working on

    • AatubeOP
      link
      fedilink
      127 months ago

      No, the settings already existed, they just forgot to expose it

      • @malloc
        link
        English
        37 months ago

        That makes sense now. Thanks for extra context.

        • AatubeOP
          link
          fedilink
          47 months ago

          (Here’s some more context: The button is in the UI, but it exposed a completely different function than advertised, probably due to being generated from copy-and-pasting the previous button)

    • @[email protected]
      link
      fedilink
      57 months ago

      Yeah without more context it just seemed they were renaming the setting as part of a larger diff

      • AatubeOP
        link
        fedilink
        17 months ago

        It says “1 changed file…” at the top

    • Victor
      link
      47 months ago

      I’m not convinced that’s it, but I don’t know enough to explain why.