CUNY Graduate Center - Project Tracking System: Issueshttps://redmine.gc.cuny.edu/https://redmine.gc.cuny.edu/favicon.ico2024-03-19T15:34:39ZCUNY Graduate Center - Project Tracking System
Redmine CUNY Academic Commons - Feature #20013 (New): Support for Commons-provided cover images in group ...https://redmine.gc.cuny.edu/issues/200132024-03-19T15:34:39ZBoone Gorgesboone@gorg.es
<p>See <a class="issue tracker-2 status-1 priority-4 priority-default" title="Feature: Updating Group/Site Creation and Completion pages (New)" href="https://redmine.gc.cuny.edu/issues/19724">#19724</a>. This ticket is a placeholder for now, but once the 2.4.0 branch is set up so that cover image support is unlocked during group creation/editing, I'll ask Jeremy to begin taking a look at how we can integrate his work from the CV creation flow into the group creation/editing flow.</p> CUNY Academic Commons - Bug #19944 (New): Dealing with BuddyPress 12.x upgrade, pt. 2https://redmine.gc.cuny.edu/issues/199442024-03-11T22:29:12ZRaymond Hoh
<p>Follow-up to <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Dealing with BuddyPress 12.x upgrade (Resolved)" href="https://redmine.gc.cuny.edu/issues/19493">#19493</a>. Just wanted to make sure my comments in <a class="external" href="https://redmine.gc.cuny.edu/issues/19493#note-10">https://redmine.gc.cuny.edu/issues/19493#note-10</a> are brought forward to a future release:</p>
<blockquote>
<p>For more pronounced changes, I created a new branch -- <a href="https://github.com/cuny-academic-commons/cac/compare/bp12...bp12-part2" class="external">bp12-part2</a> -- that namely switches out all BuddyPress 12 deprecated function calls. We can roll that out in a later maintenance release.</p>
<p>I've been testing <code>bp12-part2</code> with BP Classic <em>deactivated</em> and with <code>BP_IGNORE_DEPRECATED</code> set to <code>true</code> and these are some issues I encountered:</p>
<ul>
<li>BP Event Organiser needs some minor adjustments to account for group subnav registration: <a class="external" href="https://github.com/cuny-academic-commons/bp-event-organiser/compare/1.2.x...bp12#diff-b0397cc71e95a79af968e8dadb7b8e525085f462c5a88ac11c8bdb86ecd7173d">https://github.com/cuny-academic-commons/bp-event-organiser/compare/1.2.x...bp12#diff-b0397cc71e95a79af968e8dadb7b8e525085f462c5a88ac11c8bdb86ecd7173d</a></li>
<li>bbPress does <a href="https://github.com/bbpress/bbPress/blob/4178cbf665674c128c928fabdc0322badacad06b/src/includes/extend/buddypress/groups.php#L140-L157" class="external">some conditional checks</a> that needs to be delayed to the <code>'bp_parse_query'</code> hook. This fixes an issue with the bbPress stylesheet not loading on group forum pages I mentioned above in <a href="https://redmine.gc.cuny.edu/issues/19493#note-4" class="external">comment 4</a> amongst other permission-related cap overrides for the current user.</li>
<li>I need Git permissions to the <code>tapor-client</code> repo so I can push changes and pin a new version for our <code>cac</code> repo.</li>
</ul>
<p>About the bbPress load order issue:</p>
<blockquote>
<p>Furthermore, when running bp-classic, I had to do some additional work to ensure that bbPress's BP compatibility layer is loaded in time, and is loaded only once. Like the approach above, it's pretty hacky (requiring a static variable to prevent double-loading) but it basically works. Here's the full patch: <a class="external" href="https://github.com/openlab-at-city-tech/openlab/commit/27f2a68354550295d85f0f3c5aaafc55004268f0">https://github.com/openlab-at-city-tech/openlab/commit/27f2a68354550295d85f0f3c5aaafc55004268f0</a></p>
</blockquote>
<p>I tackled this a little differently in my bbPress commit. See <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-0f345898d2c7666be77acc95ec88e90e98a407644a4ce5431b3fb96d06749039">https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-0f345898d2c7666be77acc95ec88e90e98a407644a4ce5431b3fb96d06749039</a> and the constructor and includes() portion of <a class="external" href="https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-a8d1ad6e40805411d0e557ee641e9d47c6cc3d0f17ef107f22ac729255439a5a">https://github.com/cuny-academic-commons/cac/commit/e2145f2462d3d2cbe2f16de914e7875c48b6e1cc#diff-a8d1ad6e40805411d0e557ee641e9d47c6cc3d0f17ef107f22ac729255439a5a</a> . I think the constructor changes might fix your double load order issue. Can you do some tests and see if this works for you, Boone?</p>
</blockquote>
<p>This doesn't need to go in tomorrow's 2.3.4 release. Let's put this in 2.3.5 or later.</p> CUNY Academic Commons - Design/UX #19624 (New): Mobile Menu Tweakshttps://redmine.gc.cuny.edu/issues/196242024-01-29T22:23:58ZSara Cannon
<p>The Mobile menu can use a few tweaks. We can use the User Avatar instead of the User Icon and adjust the carats' alignment. (they seem to be at an awkward spot/angle right now)</p>
<p>Designs:<br /><img src="https://redmine.gc.cuny.edu/attachments/download/27376/Screenshot%202024-01-29%20at%203.50.13%20PM.png" alt="" /></p>
<p>The current menu with the weird carats alignment:<br /><img src="https://redmine.gc.cuny.edu/attachments/download/27377/Current%20Menu.png" alt="" /></p> CUNY Academic Commons - Design/UX #19604 (New): CV Mobile Refinementhttps://redmine.gc.cuny.edu/issues/196042024-01-25T22:51:36ZSara Cannon
<p>The Current CV Mobile view can use some design refinement. The current view has some alignment issues where the CV menu creates a scroll-to-the right situation.</p>
<p>I did a mobile comp that cleaned up the layout. Note the CV edit menu turns the "View CV" text into an Icon. This is used in WordPress Core.</p>
<p>The image credit is way too prominent, so I moved it down to the footer.</p>
<p>See the below Screenshot that shows what the mobile view is like currently, and what it could look like with a bit of cleaning up.</p>
<p><img src="https://redmine.gc.cuny.edu/attachments/download/27356/Screenshot%202024-01-25%20at%204.21.35%20PM.png" alt="" /></p>
<p>The content gutters are 16px - 4 col. See Grid Guide Below:<br /><img src="https://redmine.gc.cuny.edu/attachments/download/27357/Screenshot%202024-01-25%20at%204.21.12%20PM.png" alt="" /></p> CUNY Academic Commons - Feature #19592 (New): Groups Page Re-Designhttps://redmine.gc.cuny.edu/issues/195922024-01-24T22:41:02ZSara Cannon
<p>Below are the initial designs for the group page Redesign. I still need to do admin settings as well as the member invite modal but I thought I'd post these here to get us started.</p>
<p><img src="https://redmine.gc.cuny.edu/attachments/download/27335/Groups%20-%20Logged%20In%20-%20Registered%20(Home).png" alt="" /><br /><img src="https://redmine.gc.cuny.edu/attachments/download/27336/Groups%20-%20Logged%20In%20-%20Registered%20(Events).png" alt="" /><br /><img src="https://redmine.gc.cuny.edu/attachments/download/27337/Groups%20-%20Logged%20In%20-%20Registered%20(Library).png" alt="" /><br /><img src="https://redmine.gc.cuny.edu/attachments/download/27338/Groups%20-%20Logged%20In%20-%20Registered%20(Forums).png" alt="" /><br /><img src="https://redmine.gc.cuny.edu/attachments/download/27339/Groups%20-%20Logged%20In%20-%20Registered%20(Forums%20-%20Single).png" alt="" /></p> CUNY Academic Commons - Bug #19247 (New): CV editor scripts not loaded in Dashboard > CV > [edit]https://redmine.gc.cuny.edu/issues/192472023-11-10T18:43:40ZBoone Gorgesboone@gorg.es
<p>Jeremy, this is probably why you were not able to edit CVs in the Dashboard. It appears that the cac-cv-editor script is never loaded in the Dashboard. The script is registered in Front\Editor::setup_editor(), but it seems like this is only invoked in PostType\CV\filter_template(). Could you have a look?</p> CUNY Academic Commons - Feature #19131 (New): Add DBLP to Commons profilehttps://redmine.gc.cuny.edu/issues/191312023-10-30T16:11:02ZRaffi Khatchadourianraffi.khatchadourian@hunter.cuny.edu
<p>Please add a DBLP field (e.g., <a class="external" href="https://dblp.org/pid/42/3722.html">https://dblp.org/pid/42/3722.html</a>) to the Commons profile (e.g., <a class="external" href="https://commons.gc.cuny.edu/members/khatchad/profile/edit/group/1">https://commons.gc.cuny.edu/members/khatchad/profile/edit/group/1</a>). It would be similar to the "Google Scholar" field that is already present. Thanks!</p> CUNY Academic Commons - Bug #18213 (New): Google Analytics 4 migration documentation for usershttps://redmine.gc.cuny.edu/issues/182132023-05-12T15:40:30ZBoone Gorgesboone@gorg.es
<p>Individual sites on the Commons can have their own Google Analytics tracking. See <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: StatPress performance & migrate to per-site Google Analytics (Resolved)" href="https://redmine.gc.cuny.edu/issues/10722">#10722</a>. Since <a class="issue tracker-3 status-5 priority-4 priority-default closed" title="Support: Google Analytics 4 tagging (Resolved)" href="https://redmine.gc.cuny.edu/issues/13511">#13511</a>, we have supported Google Analytics 4 in addition to the legacy Universal Analytics. In Summer 2023, Google will be phasing out UA, so we should make some changes on the Commons:</p>
<p>1. Amend our help documentation to indicate that GA4 is the proper system to use, and to guide people on how to migrate their legacy UA accounts to GA4. Those users will then need to be directed on how to enter their new GA4 identifier in place of the old UA- identifier on their Commons sites.<br />2. We should change the language on the Commons dashboard panel so that it no longer indicates that both UA and GA4 are supported.<br />3. Scott, if you think it makes sense, we could build a system to build a list of sites using the old UA IDs. We can then either (a) email them, or (b) have something in the Commons (like an admin notice?) telling them to make the necessary changes, or (c) something else or some combination of these.</p>
<p>Scott, what do you think?</p> CUNY Academic Commons - Feature #18194 (New): Migration routine for CVshttps://redmine.gc.cuny.edu/issues/181942023-05-09T17:38:21ZBoone Gorgesboone@gorg.es
<p>When we introduce the new CV feature <a class="issue tracker-2 status-5 priority-4 priority-default closed child" title="Feature: CV Editing and Publishing (Resolved)" href="https://redmine.gc.cuny.edu/issues/17768">#17768</a> we will need to migrate over existing user data. Some initial thoughts for discussion:</p>
<p>- We should not create CVs for all existing users. How do we decide? I propose that we should only create them for users who have at least one "bottom" section filled in - Education, Positions, Publications, etc. Does anyone else have suggestions for how this should work?<br />- We'll have to build block markup using a pipeline like <code>serialize_blocks()</code>. This is likely to be subject to all sorts of issues with character encoding, etc, so we'll have to find some outlier profiles to use for testing. Ones with lots of fields built in, etc.<br />- Should old data - ie the BP profile data - be deleted or kept? It depends in part on whether we will continue to use the BP profile data. I'm thinking in part of directory search. If we do this, we'll need a mechanism in the regular CV save routine that syncs the relevant fields to BP's profile data system.</p>
<p>The migrator will have to be one of the last things built, since it'll depend on the specifics of the CV block implementation. See <a class="issue tracker-2 status-5 priority-4 priority-default closed parent" title="Feature: CV "top" section (Resolved)" href="https://redmine.gc.cuny.edu/issues/18192">#18192</a>, <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: CV "bottom" section (Resolved)" href="https://redmine.gc.cuny.edu/issues/18193">#18193</a>.</p> CUNY Academic Commons - Bug #17712 (New): Event Directory issueshttps://redmine.gc.cuny.edu/issues/177122023-02-22T22:05:04ZRaymond Hoh
<p>Just salvaging a few things brought up by Laurie and Colin in a recently-deleted ticket.</p>
<hr />
<p>Laurie wrote:</p>
<blockquote>
<p>I created a text event in a private group but my "all upcoming" events tab reads zero "0" , see screenshot. Shouldn't the # counts described the following:</p>
<p>"all upcoming" = # of public + events you created in private groups on the commons</p>
<p>"My upcoming" = # of events in private and public groups you are a part of, and events you created</p>
</blockquote>
<hr />
<p>Confirmed about the "All Upcoming" count not including private events that the user authored. Will look into this.</p>
<hr />
<p>Colin wrote:</p>
<blockquote>
<p>Speaking of... there are no upcoming or visible events on the /events/ page, so anything we can do to encourage more event creation (including publishing more ourselves) would help to make that top level page more vibrant.</p>
</blockquote>
<hr />
<p>Maybe we can add a message when there are no upcoming events.</p>
<p>For logged-in users:</p>
<blockquote>
<p>There are no upcoming events. Feel free to [add your upcoming event to the calendar](LINK). You can also view prior events [here](LINK TO PREVIOUS MONTH'S EVENTS).</p>
</blockquote>
<p>For logged-out users:</p>
<blockquote>
<p>There are no public, upcoming events. You can view prior events [here](LINK TO PREVIOUS MONTH's EVENTS).</p>
</blockquote> CUNY Academic Commons - Bug #17587 (New): Recurring event issueshttps://redmine.gc.cuny.edu/issues/175872023-01-31T07:16:53ZRaymond Hoh
<p>I came across a few bugs while testing events:</p>
<ul>
<li>Dates for recurring events are not displayed correctly in list view. The first event date is shown instead of the actual recurring event's date.</li>
<li>Recurring date info is missing on single event pages. For a point of reference, a recurring event page looks like this in Event Organiser: <a class="external" href="http://demo.wp-event-organiser.com/events/event/fortnightly-event/">http://demo.wp-event-organiser.com/events/event/fortnightly-event/</a>. Do we want to port over the <code>"This event is running from X until Y. It is next occurring on Z"</code> line or the "Upcoming Dates" section? (Also, venue map is missing on our single event page as well.)</li>
</ul>
<p>Will add a fix for the former, but could use some feedback on the latter.</p> CUNY Academic Commons - Bug #17551 (New): Events Calendar viewhttps://redmine.gc.cuny.edu/issues/175512023-01-24T21:44:09Zscott vothscott.voth@verizon.net
<p>When I switch from "List View" to "Calendar View" I get a "Loading" icon spinning around. In debugger seems like some CORS issue.</p>
<p>I wanted to get some screenshots of December events, so I "back arrowed" to December. When I go back to list view everything is fine. But I am still in January, not December.</p> CUNY Academic Commons - Feature #15194 (New): PHPCS sniff for un-restored switch_to_blog() callshttps://redmine.gc.cuny.edu/issues/151942022-01-14T20:15:06ZBoone Gorgesboone@gorg.es
<p>When <code>switch_to_blog()</code> is called but not properly "closed" via <code>restore_current_blog()</code>, havoc ensues. The most recent instance of this is #15170.</p>
<p>We should introduce a PHPCS sniff to our custom ruleset that looks for <code>switch_to_blog()</code>, and then ensures that all code paths out of the method have a <code>restore_current_blog()</code> before returning.</p> CUNY Academic Commons - Support #11496 (New): Replace Twitter Icon on Member Portfolio page https://redmine.gc.cuny.edu/issues/114962019-05-29T12:16:18Zscott vothscott.voth@verizon.net
<p>A member noticed that the twitter icon used on the member portfolio page is out of date. Is it possible to switch to the new one? <a class="external" href="https://about.twitter.com/en_us/company/brand-resources.html">https://about.twitter.com/en_us/company/brand-resources.html</a><br />(The old twitter logo looks a bit like the current tumblr icon to him.)</p> CUNY Academic Commons - Publicity #3506 (New): Prepare 1.7 email messaginghttps://redmine.gc.cuny.edu/issues/35062014-10-01T16:36:42ZMicki Kaufman
<p>Write up a lovely email describing the revolutionary new 'My Commons' page, and the rest of the delightful 1.7 feature set!</p>