I already forked the ui-repo, found the correct file to fix it and even found the place where I should add a query to the search form to search only for communities if you start searching from the communities page.

But then I saw that this change is already done in that file. I double checked the HTML on my instance and there it’s broken. I checked the history of the page and see it has been fixed 18 hours before I started looking into it.

All my hard work for nothing.

  • scottyjoe9@sh.itjust.works
    link
    fedilink
    English
    arrow-up
    2
    ·
    1 year ago

    Plenty more bugs to fix, don’t give up being a contributor!

    My process is:

    1. Check for an issue that describes the bug you’re seeing.
    2. If there isn’t one, create one.
    3. If you feel like it’ll be snapped up because it’s a relatively easy fix, comment on the issue saying you’ll give it a go.
    4. Fix it!

    There is still a chance someone might swoop in and fix it but this lowers the chances and it’s good to get into the habit of creating and linking Issues to your PRs.

  • lml@remy.city
    link
    fedilink
    arrow-up
    2
    ·
    1 year ago

    At least you got yourself into the contributing mindset. Tackle the next issue!

  • Jeena@jemmy.jeena.netOP
    link
    fedilink
    English
    arrow-up
    1
    ·
    1 year ago

    I have another bug which is bothering me. When the site settings are set to "Registration Mode": "Close Registration", the Sign Up link still shows up and the route still sends you to the Sign Up form. You can fill in everything there but then will get a infinite spinner.

    I wanted to at least remove the link at the top when sign up is closed. But obviously it’s not as easy as changing a static HTML as in the other bug.