No internet connection
  1. Home
  2. Issues

How can I set categories / all questions to be publicly viewable?

By Florian Engel @moreonion
    2020-05-25 15:37:21.826Z

    I've set up an account and changed the "admin" section here to make sure it doesn't require login to view questions.

    Then I set up a test-question with this URL:
    https://impact-stack.talkyard.net/-9

    But when I try to access the page without an open user session I still get the prompt to log in. Am I missing something or is this a bug?

    Reply4 Likes
    • 11 replies
    1. KajMagnus @KajMagnus2020-05-26 04:44:52.831Z

      Hi Florian, this looks odd, I'll investigate. Might be a bug. Thanks for reporting

      Reply
      1. KajMagnus @KajMagnus2020-05-26 05:58:00.257Z

        Seems there's a bug yes: Approve users also enables Login required, although not visible in the admin interface.

        As a temporary workaround, you can uncheck Approve users. Then wait for me to fix the bug, and thereafter you can enable Approve users again (or I can do that for you if you want?).

        ( Details: Maybe next week, I'll do this: A data migration that enables Login required for all Talkyard sites with Approve users enabled — so it becomes visible in the config settings & admin interface, how those sites currently behave.

        And I'll fix the bug, so that Approve users thereafter will not indirectly enable Login required. And, because of the above-mentioned data migration, this won't affect any site — however if they later choose to un-check Login required, then, that'll work as expected.

        Hmm, I've even added a source code comment about this long ago:

                Setting2(props, { type: 'checkbox', label: "Approve users", id: 'e_ApproveUsersCB',
          className: 'e_A_Ss_S-ApproveUsersCB',
          help: "New users need to be approved by staff before they can do anything more " +
   ——>        "than just reading.",  // docs BUG: currently won't get access at all [2KZMQ5]

        )

        Reply
        1. MFlorian Engel @moreonion
            2020-05-26 09:20:36.528Z

            Thanks, I've changed the setting and now it works :)

            Reply
            1. In reply toKajMagnus⬆:
              JPhilip @jakob
                2022-06-16 19:03:17.782Z

                Hey @KajMagnus ,

                We just ran into the same issue, running Talkyard v0.2022.10.

                Is there still the plan to fix this issue?

                Thanks a lot!
                Philip

                Reply1 Like
                1. KajMagnus @KajMagnus2022-06-20 12:22:03.509Z

                  Yes, but I forgot. What if you click Like on the Original Post? I just changed the sort order in the Issues category to "Popular first" (eh, "Most problematic" first would be a better title).

                  So if you Like vote it, it'll get listed higher up and I'll remember :- )

                  Reply1 Like
                  1. SStefan Herzog @stefanherzog
                      2022-06-21 14:04:58.616Z

                      Great! I have the same issue like @jakob . I've liked the original post.

                      Reply
                  2. In reply toKajMagnus⬆:
                    @Mr.Nobody
                      2023-08-23 20:30:04.318Z

                      KajMagnus: "Seems there's a bug yes: Approve users also enables Login required, although not visible in the admin interface.

                      As a temporary workaround, you can uncheck Approve users. Then wait for me to fix the bug, and thereafter you can enable Approve users again (or I can do that for you if you want?).

                      ( Details: Maybe next week, I'll do this: A data migration that enables Login required for all Talkyard sites with Approve users enabled — so it becomes visible in the config settings & admin interface, how those sites currently behave.

                      And I'll fix the bug, so that Approve users thereafter will not indirectly enable Login required. And, because of the above-mentioned data migration, this won't affect any site — however if they later choose to un-check Login required, then, that'll work as expected.

                      Hmm, I've even added a source code comment about this long ago:"

                      Setting2(props, { type: 'checkbox', label: "Approve users", id: 'e_ApproveUsersCB',
className: 'e_A_Ss_S-ApproveUsersCB',
help: "New users need to be approved by staff before they can do anything more " +
 ——>        "than just reading.",  // docs BUG: currently won't get access at all [2KZMQ5]```

                      The bug is still there. it just happened to me and I had to disable user moderation to fix it.

                      Reply
                      1. KajMagnus @KajMagnus2023-08-24 13:52:19.058Z

                        Sorry, seems I forgot. I'll have a look later this week (hopefully for real this time)

                        Reply1 Like
                        1. KajMagnus @KajMagnus2023-08-27 20:07:38.165Z

                          @Mr.Nobody — next week :- (. I've been trying to finish tags-with-values :- )   and it seems tags-with-values works now, ... "Only" code review and auto tests remain

                          Reply1 Like
                          1. @Mr.Nobody
                              2023-08-28 17:23:22.608Z

                              all good, this is not a huge issue since there is a workaround.

                              Reply
                    • In reply tomoreonion⬆:
                      KajMagnus @KajMagnus2023-10-16 11:03:51.076Z

                      Now it's soon time to fix this, at last :- ) Tags-with-values now implemented, and I tagged this with priority: 5 (which will mean "Important (and not urgent)"), so I'm less likely to forget again. ... Now I can close this browser tab

                      Reply
                      1. Progress
                        with handling this problem
                      2. @KajMagnus marked this topic as Planned 2020-05-26 06:00:27.704Z.
                      Reply (discussion)Add progress note