samtar opened https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/56

Show codex dialog if user is not logged in

nb. currently missing messages in WikimediaMessages

I wonder if a dialog is the right approach here. For starters, Codex dialogs evidently are supposed to be dismissable. They also state dialogs should not be used when "the information can be displayed inline within the main interface", which is the case here:

For the gadget

• We can rely on the fallback text that currently lives at [[Community Requests/Intake]]. Just add the text at the top with a link to login.
• We can use user group CSS to show different messaging depending on if you're logged in.
  • Wrap text you want only shown to anonymous users with <div class="anonymous-show">
  • Wrap text to be shown only for logged-in users with <div class="user-show">.
• Change the gadget definition to have rights=editmyusercss so that the gadget only loads for logged-in users. This is better for performance.
• We can use the title denylist to entirely disable editing of subpages of [[Community Requests]] for anonymous users, and show custom messaging.

For the extension (later)

Even if we kept with the wikitext-based fallbacks (TBD), we should still have a safety net similar to what this task is asking for, but I don't think we should load the form at all for logged in users, if they aren't meant to be able to use it. Instead, we'd likely have wish editing on a Special page, so we just put the normal permission checks behind it with some custom messaging if we want, similar to i.e. https://ee.wikipedia.org/wiki/Special:Block (assuming no one reading this is an admin there).

musikanimal opened https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/57

DeploymentCommand: add user-group-css task for user group CSS

samtar merged https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/57

DeploymentCommand: add user-group-css task for user group CSS

@JWheeler-WMF any opposition to what I'm proposing at T363221#9849068? The short of is, instead of showing the form with a dialog (not recommended by Codex), we show the message before we even load the Vue application. In fact, the gadget should not run at all for logged out users, as it is only used for editing. This is also better for performance, since we don't want to load the giant JS payload for users who won't be able to make use of it.

So a logged out user would visit i.e. [[Community Requests/Intake]] and see a message like "Sorry, Community Requests requires you be logged in to submit ideas" (or whatever the language is following the naming decision), and a link to login.

That sound okay?

Works for me!

musikanimal opened https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/69

Don't load gadget for logged-out users and show message to log in

musikanimal closed https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/56

Show codex dialog if user is not logged in

tstarling merged https://gitlab.wikimedia.org/repos/commtech/wishlist-intake/-/merge_requests/69

Don't load gadget for logged-out users and show message to log in

@MusikAnimal Login vs logged out is working as designed. I will move this to Done. Thanks for all your work!

Test Result - Patchdemo

Status: ✅ PASS
Environment: Patchdemo
OS: macOS Sonoma 14.5
Browser: Chrome 126
Device: MBA
Emulated Device: NA

Test Artifact(s):

Test Steps

Test Case 1: Display log-in message for logged-out users

1. Navigate to the wish submission form without being logged in (https://patchdemo.wmflabs.org/wikis/780dec507c/wiki/Community_Requests/Intake)
2. Observe the page content.

✅ AC1: Verify a message prompts the user to log in and does not load.

Test Case 2: Form availability for logged-in users

1. Log in to the site (https://patchdemo.wmflabs.org/wikis/780dec507c/wiki/Community_Requests/Intake)
2. Navigate to the wish submission form.

✅ AC2: Verify the form loads correctly.