• Blaze (he/him)@lemmy.zipOP
      link
      fedilink
      English
      arrow-up
      15
      ·
      6 days ago

      There was a revival attempt at !trendingcommunities@lemmy.cafe , but it was short lived.

      That probably shows a trend: those type of tools need to be built-in into the software, relying on Raspberry Pies, scripts and external tools to provide such features will likely lead to them being abandoned

      • Coelacanth@feddit.nu
        link
        fedilink
        English
        arrow-up
        7
        ·
        6 days ago

        Oh I missed that somehow! I’ll subscribe just in case, but I agree with you that the functionality should ideally be built in to the software.

      • db0@lemmy.dbzer0.com
        link
        fedilink
        English
        arrow-up
        2
        ·
        6 days ago

        I don’t think piefed has the same API has lemmy, so no. It’s been something I wanted to onboard (the piefed API) ,but I haven’t fouind the time.

        • SatyrSack@quokk.au
          link
          fedilink
          English
          arrow-up
          2
          ·
          6 days ago

          I do see there is code to throw an error when it detects an instance is not Lemmy, so it definitely doesn’t work out-of-the-box. But the APIs for each are supposedly similar to each other, so it fortunately should not take any major rewrite to support both in the same wrapper.

            • wjs018@piefed.social
              link
              fedilink
              English
              arrow-up
              4
              ·
              6 days ago

              Like Blaze mentioned, the api is a bit of a work in progress and we are revamping the swagger. The version of the api spec you have there (freamon’s) is manually created and can have some inaccuracies because of that. We are in the process of making the piefed api self-documenting so that inputs/outputs are validated against the schema, but there are a lot of endpoints still to go in that effort.

              Some of those revamped endpoints are going to be coming in the 1.2 release. For the self-documented endpoints that have been done so far, you can check the swagger on crust (the dev instance running right off the main branch).

              To keep up with this effort as I am working through it, I have been making posts to !piefed_api@piefed.social.


              On a personal note, I would kill for a pythorhead-but-for-piefed to exist. I pretty extensively use pythorhead in my lemmy communities now, so I am hopeful that we can get the api in a stable enough state to enable tools like this to exist in the near future.

              • db0@lemmy.dbzer0.com
                link
                fedilink
                English
                arrow-up
                3
                ·
                6 days ago

                Yeah I would like to support it as well. I’ll wait until the API stabilizes though to avoid more work

              • freamon@preferred.social
                link
                fedilink
                English
                arrow-up
                3
                ·
                5 days ago

                I’ll just remove the ‘freamon’ one when the auto-generated one is up to date.

                The manually-generated one had 5 missing routes, which I’ve since added.

                The auto-generated one at crust has about 48 missing routes. It’s the right approach, and I’ll help out with it when I can, but - for now at least - it makes no sense to redirect people to it (either automatically or via a comment).


                Some thoughts for @wjs018@piefed.social

                /site/instance_chooser probably doesn’t need to be a route. It’s just the data format returned by /site/instance_chooser_search. As a route, it’s returning the instance info for the site you’re querying, so if you want to keep it as a route, it should probably be called /site/instance_info or something.

                In the query for /site/instance_chooser_search, nsfw and newbie are both booleans. With the rest of the API, these are sent as ‘true’ or ‘false’, but they are ‘yes’ and ‘no’ for this route. The newbie query should probably be newbie_friendly In the response, monthsmonitored should probably be months_monitored

                There’s no way to exclude communities for the response to /topic/list and /feed/list: If you don’t put ‘include_communities’ in the query, it’s defaults to True, but if you put ‘include_communities=false’ in the query it ends up being True also (because the word ‘include_communities’ is in the data).

                • wjs018@piefed.social
                  link
                  fedilink
                  English
                  arrow-up
                  3
                  ·
                  5 days ago

                  TIL that rimu added instance chooser endpoints. I’ll take a look at those before 1.2 gets shipped. Same with the topic and feed endpoints. The easiest time to make changes to them is now before they get more widely used.

                  I have about 10 more endpoints (working through /user right now) I have finished transitioning in a dev branch right now, so we are making progress.

      • Coelacanth@feddit.nu
        link
        fedilink
        English
        arrow-up
        1
        ·
        6 days ago

        No idea, it just stopped working. It already didn’t work with Piefed communities I think, maybe it was taken down for maintenance and never put back up?