CUNY Graduate Center - Project Tracking System: Issueshttps://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2024-03-14T15:49:10ZCUNY Graduate Center - Project Tracking System
Redmine CUNY Academic Commons - Feature #19980 (Assigned): Better duplicate-notification prevention for b...https://redmine.gc.cuny.edu/issues/199802024-03-14T15:49:10ZBoone Gorgesboone@gorg.es
<p>bp-multiple-forum-post was written before BPGES started using an asynchronous queue technique for sending notifications. As a result, its mechanism for preventing duplicate notifications - which involves a runtime global - is not effective. Instead, we should prevent BPGES from entering a record into the queue table when we see that the user already has the activity item in his/her queue.</p>
<p>This in itself will not be enough. If you post to A and cross-post to B, you could have a situation where:<br />1. The item is queued with respect to group A, and bp-multiple-forum-post schedules a task to create the crossposts<br />2. BPGES runs its "immediate" queue and your A notification goes out. It's then deleted from the queue.<br />3. Later, bp-multiple-forum-post creates its crossposts. Since the A notification has been cleared from the queue, the check described above won't be found.</p>
<p>Modifying BPGES to keep a persistent queue seems like a bad idea - it'd mean introducing a 'status' column, and it'd mean keeping items around forever, unless we had a cleanup routine.</p>
<p>I guess the alternative is to keep track in some other way of users who have received notifications (or, better still, users for whom a BPGES notification has been queued) on a cross-posted item. There's no beautiful way to do this. I guess maybe a piece of postmeta that lives with the original post, which will have to be checked when bp-multiple-forum-post creates the crossposts.</p> CUNY Academic Commons - Bug #18016 (Hold): ml-slider get_plugins() call causes performance issueshttps://redmine.gc.cuny.edu/issues/180162023-04-12T16:57:56ZBoone Gorgesboone@gorg.es
<p>See discussion at <a class="external" href="https://redmine.gc.cuny.edu/issues/15767#note-2">https://redmine.gc.cuny.edu/issues/15767#note-2</a> and follow-up. The ml-slider folks have promised to fix the issue, but have not done so. <a class="external" href="https://wordpress.org/support/topic/call-to-get_plugins-causing-performance-problems/">https://wordpress.org/support/topic/call-to-get_plugins-causing-performance-problems/</a></p>
<p>Until they do, we will need to patch ml-slider each time it's updated. This happens next in the 2.1.5 release.</p>
<p>We should also have some sort of system for notifying the release manager (ie me) whenever this patch needs to be applied. Looking at GitHub Actions results is already part of my workflow, which helps me catch invalid syntax and other items. But this one is kinda hard to catch with static analysis tools. Instead, I'm thinking I'll add something to the wp-cli-cac script that I use for bulk plugin updates, something that throws a huge notice about plugins that need manual attention after they've received an update.</p> CUNY Academic Commons - Feature #16092 (Hold): Don't show main site in Site search resultshttps://redmine.gc.cuny.edu/issues/160922022-05-11T22:40:36ZBoone Gorgesboone@gorg.es
<p>From <a class="external" href="https://redmine.gc.cuny.edu/issues/15261#note-31">https://redmine.gc.cuny.edu/issues/15261#note-31</a>:</p>
<blockquote>
<p>- The CAC home site seems to appear in search results. Can we filter that out?</p>
</blockquote>
<p>I don't know whether this is something we actually want to do. If there's general agreement, please let me know and I'll look into technical feasibility.</p> CUNY Academic Commons - Bug #12436 (Assigned): Nightly system downtimehttps://redmine.gc.cuny.edu/issues/124362020-02-18T16:21:55ZBoone Gorgesboone@gorg.es
<p>I'm opening this ticket to track recent outages on the Commons. A number of us receive automated notices when the database server is forced to reboot. Beginning roughly one month ago, we started getting these notices several times per week. These notifications indicate when the DB server is forced to reboot, so they indicate at best the end of a problematic period. I receive additional notifications when requests begin to take an inordinately long time, which serves as an indicator of when the incidents begin.</p>
<p>I've been keeping track of specifics over the last few weeks, and I've discerned the following patterns:</p>
<p>- The most common downtime is just after 05:00 UTC (midnight EST, UTC-5), with my incident reports rolling in sometimes around 12:03am and sometimes around 12:07am.<br />- Occasionally, the incidents have begun an hour or two earlier, shortly after 03:00 or 04:00 UTC.<br />- Incidents seem always to begin several minutes after the hour<br />- Reboots usually take place between 3 and 6 minutes after the beginning of the incident<br />- On some occasions, the reboots don't seem to fix the underlying issue, and another cycle of slow requests + db reboots immediately follows.<br />- Sometimes this'll happen a few nights in a row, while sometimes the site will go a few days without any notifications.</p>
<p>All of this strongly suggests that the problem is with an automated cron task, specifically one that takes place around midnight. I've begun to do an analysis of the tasks scheduled for around this time, cross-referencing with the logs. Ideally, we'd be able to narrow down the culprit by identifying the last cron task that begins just before the outages. This is not possible, for a couple reasons: First, the performance issues may only kick in a minute or two after the task begins running (as the system's resources are gradually used up). Second, the Cavalcade logs don't natively keep track of when a task begins running, but only when it finishes (see <a class="external" href="https://github.com/humanmade/Cavalcade-Runner/blob/master/inc/class-runner.php#L377">https://github.com/humanmade/Cavalcade-Runner/blob/master/inc/class-runner.php#L377</a>).</p>
<p>So the best we can do is to make some educated guesses. I'll follow up in a comment with initial thoughts.</p> AD/O365 Transition from NonMatric to Matriculated Students - Feature #12062 (In Progress): creat...https://redmine.gc.cuny.edu/issues/120622019-11-12T20:50:21ZEmilio RodriguezCUNY Academic Commons - Feature #11945 (Reporter Feedback): Add Comments bubble to Reckoning viewshttps://redmine.gc.cuny.edu/issues/119452019-10-04T18:53:28ZBoone Gorgesboone@gorg.es
<p>Splitting off from <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Integration between WP Grade Comments and Reckoning (Resolved)" href="https://redmine.gc.cuny.edu/issues/11937">#11937</a>:</p>
<blockquote>
<p>The comment bubble links to a dashboard page with all the comments on a particular post (screenshot). I was suggesting integrating this comment bubble into reckoning for each post. So along with the post name, post category (to be added), dated posted, and grade (to be integrated), you could also see the dynamic comment bubble and when clicked, be taken to the dashboard page of comments on post (screenshot).</p>
<p>If the comment bubble is integrated into reckoning, when viewing reckoning with post information, a prof could easily navigate to view all the comments on a particular post, including their own, which maybe be associated with the grade that is visible in reckoning.</p>
</blockquote> CUNY Academic Commons - Support #11883 (New): Need Embedding Help Page Update (Tableau)https://redmine.gc.cuny.edu/issues/118832019-09-19T21:43:37ZAnthony Wheeler
<p>Hi Scott,</p>
<p>So I was recently trying to embed a Tableau visualization into a Commons site when I ran into a couple of roadblocks. I went to our Help Page for embedding from other websites, and it was helpful, but something we may want to add is that when copying the share link over from Tableau, you have to make sure you copy the link under "Original View" <br /><img src="https://redmine.gc.cuny.edu/attachments/download/12443/Screen%20Shot%202019-09-19%20at%205.40.11%20PM.png" alt="" /></p>
<p>It was only after that discovery that the visualization was fully functioning within the Commons. I attached (or tried to attach) a screenshot to hopefully show you what I'm talking about.</p>
<p>Thanks!</p> CUNY Academic Commons - Feature #11517 (Assigned): wp-accessibility plugin should not strip 'targ...https://redmine.gc.cuny.edu/issues/115172019-06-04T15:08:25ZBoone Gorgesboone@gorg.es
<p>The default setting of wp-accessibility is to strip 'target="_blank"' from links. See <a class="issue tracker-1 status-8 priority-4 priority-default closed" title="Bug: Links not opening in new tab (Duplicate)" href="https://redmine.gc.cuny.edu/issues/11512">#11512</a>, <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Support: open link in a new tab not working (Resolved)" href="https://redmine.gc.cuny.edu/issues/11002">#11002</a>.</p>
<p>In <a class="issue tracker-1 status-8 priority-4 priority-default closed" title="Bug: Links not opening in new tab (Duplicate)" href="https://redmine.gc.cuny.edu/issues/11512">#11512</a>, Gina suggests that this should not be the default behavior:</p>
<blockquote>
<p>Since the plugin appears to insert such a warning in the HTML, would it be possible to configure it so that it does not default to removing the target attribute from links? Removing the target attribute is likely to be confusing for other users of the Teaching template.</p>
</blockquote>
<p>I personally am not a fan of target="_blank". See <a class="external" href="https://redmine.gc.cuny.edu/issues/10712#note-2">https://redmine.gc.cuny.edu/issues/10712#note-2</a> and follow-up conversation. Moreover, I'm not a huge fan of messing with the default settings of third-party plugins, because it complicates updates and puts us out of step with their documentation, etc. But if others feel strongly that the current behavior is potentially confusing, it's probably possible to change the default behavior, either globally or in the case of the Teaching Template only.</p>
<p>I'm assigning this to Laurie as she's been the point person for building these templates, but general feedback is welcome.</p> CUNY Academic Commons - Feature #10380 (In Progress): Remove blacklisted pluginshttps://redmine.gc.cuny.edu/issues/103802018-09-25T14:52:36ZBoone Gorgesboone@gorg.es
<p>I'd like to do a review over the next few releases of blacklisted plugins. These are plugins that are unavailable on Dashboard > Plugins, but may still be running on existing sites. For those that are running only on a small handful of sites, it may be worth removing the plugin from the codebase altogether. This data will take some time to collect.</p>
<p>Scott, I'm copying you in case I have questions on any of them.</p> CUNY Academic Commons - Feature #5234 (Assigned): Write Unconfirmed patch for WPhttps://redmine.gc.cuny.edu/issues/52342016-02-19T04:38:17ZBoone Gorgesboone@gorg.es
<p>See <a class="external" href="https://core.trac.wordpress.org/ticket/16001">https://core.trac.wordpress.org/ticket/16001</a>, <a class="external" href="https://core.trac.wordpress.org/ticket/27145">https://core.trac.wordpress.org/ticket/27145</a>.</p>
<p>It's likely that most of the code would have to be rewritten (eg to use list tables) but the UI elements can probably be kept.</p> CUNY Academic Commons - Bug #4388 (Assigned): Repeated request for authentication.https://redmine.gc.cuny.edu/issues/43882015-08-05T19:43:46ZAlice.Lynn McMichaelalimcmichael@hotmail.com
<p>Hi, I'm building a site on the Commons, and when I click on a page or refresh one, a pop-up asking for authentication interrupts the work, often making me click OK up to 8-10 times. (It doesn't go away if I click Cancel, either). I can't tell whether this is Commons-wide or related to my account or theme, although it did not happen when I used a different Commons site yesterday</p>
<p>My site is not public yet, but the URL is amcmichael.commons.gc.cuny.edu and the theme is Make. I've had this problem repeatedly over the last couple of weeks. (See the attached screenshot for the pop-up box that I get).</p>
<p>Thank you.<br />Best,<br />Alice Lynn</p> CUNY Academic Commons - Feature #3330 (Assigned): "Commons Information" toolhttps://redmine.gc.cuny.edu/issues/33302014-07-16T17:52:38ZBoone Gorgesboone@gorg.es
<p>See <a class="issue tracker-2 status-5 priority-4 priority-default closed parent" title="Feature: My Commons (Resolved)" href="https://redmine.gc.cuny.edu/issues/2945">#2945</a> and <a class="external" href="http://redmine.gc.cuny.edu/attachments/download/1293/Commons-PersonalHomepage-Options-v3.pdf">http://redmine.gc.cuny.edu/attachments/download/1293/Commons-PersonalHomepage-Options-v3.pdf</a></p>
<p>We need a plugin that will power the Commons Information section of the profile and personal home page. Based on the wireframes and team discussions, I'm going to suggest the following for the first iteration (keeping in mind that we can add fancy stuff in the future):</p>
<p>- New custom post type: "Commons Information". Available to Editors and Administrators (current_user_can( 'edit_others_posts' ))<br />- The Commons Information "widget" won't really be a widget, but just a function that spits out some markup<br />- Widget should just show the most recent item + featured image. Mockup doesn't show titles, but I imagine we'll want them - right below where the image appears<br />- The 'show more' link at the bottom will fade the content of the widget, send an AJAX request, and show the next item (and cycle back to the first one when done). Don't show the 'view more' link on noscript.</p>
<p>Chris, feel free to chime in with additional thoughts or suggestions.</p>
<p>Dan, are you up for the challenge? :) It's a bit tricky because you'll have to develop it in isolation (you won't be able to put it in its proper place in the theme yet because that place doesn't exist yet; see <a class="issue tracker-2 status-6 priority-4 priority-default closed child" title="Feature: Rearrange columns on user pages (Rejected)" href="https://redmine.gc.cuny.edu/issues/3329">#3329</a>), but I don't think that'll be a serious problem. Let me know what you think, or if I can help get you started somehow.</p> CUNY Academic Commons - Feature #3230 (Assigned): Scripts for quicker provisioning/updating of de...https://redmine.gc.cuny.edu/issues/32302014-05-28T13:42:42ZBoone Gorgesboone@gorg.es
<p>As the Commons team grows, more people are faced with the task of creating a local enviroment that mirrors the Commons for their development work. This is currently not an easy task. At a high level, you have to:</p>
<p>- make sure that a *AMP environment is running on your computer, and is roughly compatible with the server software running on the Commons production server<br />- clone the Commons repo<br />- download and import some version of the Commons database into your own local database<br />- do a bunch of wizardry to ensure that either you can switch your local environment to self-identify as 'commons.gc.cuny.edu', or to swap out with a local dev URL like 'commons.local.dev' or 'local.cicdev.com'<br />- maybe get some representative piece of the data at wp-content/blogs.dir/</p>
<p>Each of these steps is fraught with issues about data integrity, implementation details, and various, unpleasant gotchas.</p>
<p>Let's start to work toward something a bit more organized. I'm going to suggest the following as a starting point:</p>
<p>- Let's use Vagrant for VM management, since it's pretty widely used in the WP community<br />- Pick a provisioning tool - bash, Puppet, Salt, Chef, etc - for doing environment provisioning. I don't have a ton of experience here, so if anyone has thoughts, please jump in<br />- Write the necessary provisioning scripts to mirror the Commons fairly well (keeping in mind that we don't control the Commons environment directly, and that the Commons runs RHEL, which means we'll need to use a pretty-close free alternative OS)<br />- Write some scripts to handle fetching recent versions of Commons data, including a "cleaned-up" version of production data (I'll open a separate ticket for this)</p>
<p>I think this would be a cool summer project. Dom, I'm going to assign to you for the moment, as it seems up your alley, and ask you to give your initial thoughts.</p> CUNY Academic Commons - Feature #3193 (Assigned): bbPress 2.x dynamic roles and RBEhttps://redmine.gc.cuny.edu/issues/31932014-05-02T13:49:52ZBoone Gorgesboone@gorg.es
<p>From <a class="issue tracker-2 status-5 priority-4 priority-default closed parent" title="Feature: Upgrade bbPress to 2.x (Resolved)" href="https://redmine.gc.cuny.edu/issues/3177">#3177</a> <a class="external" href="http://redmine.gc.cuny.edu/issues/3177#note-3">http://redmine.gc.cuny.edu/issues/3177#note-3</a></p>
<blockquote>
<p>One other thing to note is bbPress has the concept of dynamic user roles. At the moment, this conflicts with Reply By Email because I wanted to copy bbPress' validation schema. And since it is hard to determine these dynamic roles when the user isn't logged in, replies will fail to post. (Or, I'm just not looking in the right place ;)</p>
<p>A way around this is to set the user role automatically in bbPress 2 so a role is saved in the database.</p>
</blockquote>
<p>Ray - setting user role automatically is fine with me, as long as it doesn't cause other problems. I mention this because of <a class="external" href="https://bbpress.trac.wordpress.org/ticket/2210">https://bbpress.trac.wordpress.org/ticket/2210</a>, which is the one area of bbPress that I am familiar with :)</p>
<p>So, it could be that this ticket can be resolved with a simple settings toggle. In any case, it's a placeholder to remind us to check.</p> CUNY Academic Commons - Feature #3192 (Assigned): Customizable forum views for bbPress 2.x group ...https://redmine.gc.cuny.edu/issues/31922014-05-02T13:44:43ZBoone Gorgesboone@gorg.es
<p>From <a class="issue tracker-2 status-5 priority-4 priority-default closed parent" title="Feature: Upgrade bbPress to 2.x (Resolved)" href="https://redmine.gc.cuny.edu/issues/3177">#3177</a>:</p>
<p>Boone: "We have some customizations that allow users to modify "posts per page" and "order" when viewing a topic. It should be possible to do the same in bbPress 2.x, but it'll need to be rewritten from scratch. We may decide that this particular task could be postponed if it's going to be a lot of work."</p>
<p>Ray: " I've always wanted to write a plugin that adds a bbPress forum settings page to a BP profile. This page will have settings such as sort order and pagination. Probably a good opportunity to do so."</p>
<p>I like the idea of creating a per-user setting that lets people decide how they'd like to view forums by default. But I don't think we should lose the per-thread, ad hoc settings - it's often only in certain situations that we want to, say, view by ASC rather than DESC.</p>
<p>Ray, I'm going to pass this over to you. I've added a number of other team members to chime in on what they do and don't like about the current implementation.</p>
<p>(I'll also note that, IMHO, this is a lower-priority forums feature. If we were pressed for time, we shouldn't let this one hold up a release.)</p>