Entering "ga" always opens a user's assignments, even when they can't access them

Firepup650 · May 10, 2024, 10:53pm

This is reproducible anywhere with Discourse assign, including here. Just enter ga when not in a text box and the bind triggers, regardless if you’re allowed to do so or not.

Arkshine · May 11, 2024, 12:02am

Good catch

github.com

discourse/discourse-assign/blob/main/assets/javascripts/discourse/initializers/extend-for-assigns.js#L758


      
              pluginId: PLUGIN_ID,
          
              actions: {
                save() {
                  this.saveAttrNames.push("custom_fields");
                  this._super(...arguments);
                },
              },
            });
          
            api.addKeyboardShortcut("g a", "", { path: "/my/activity/assigned" });
          }
          
          const REGEXP_USERNAME_PREFIX = /^(assigned:)/gi;
          
          export default {
            name: "extend-for-assign",
          
            initialize(container) {
              const siteSettings = container.lookup("service:site-settings");
              if (!siteSettings.assign_enabled) {

That line might need to be moved inside a api.getCurrentUser()?.can_assign condition.

Topic		Replies	Views
Editting an assigned post re-sends the assigned mail Bug	7	679	January 10, 2019
Discourse Assign Plugin official , assign	11	37334	September 15, 2024
Assign topics to non-staff users Feature assign	27	4674	July 7, 2021
Increase assigned topic visiblity Feature assign	7	2695	November 3, 2020
Bug - possible for admin to assign PM to moderator who does not have access to the PM Bug assign	8	2408	August 14, 2020

Entering "ga" always opens a user's assignments, even when they can't access them

Related topics