Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PRJ-59 Members directory readylaunch template #4602

Open
wants to merge 252 commits into
base: release
Choose a base branch
from
Open

Conversation

bb-yudhisthir
Copy link
Contributor

Jira Issue:

https://buddyboss.atlassian.net/browse/PRJ-59

General Note

Keep all conversations related to this PR in the associated Jira issue(s). Do NOT add comment on this PR or edit this PR’s description.

Notes to Developer

  • Ensure the IDs (i.e. PROD-1) of all associated Jira issues are reference in this PR’s title
  • Ensure that you have achieved the Definition of Done before submitting for review
  • When this PR is ready for review, move the associate Jira issue(s) to “Needs Review” (or “Code Review” for Dev Tasks)

Notes to Reviewer

  • Ensure that the Definition of Done have been achieved before approving a PR
  • When this PR is approved, move the associated Jira issue(s) to “Needs QA” (or “Approved” for Dev Tasks)

KartikSuthar and others added 30 commits September 24, 2024 11:47
* release: (297 commits)
Update: Grunt
Update: Grunt string-replace
Update: Version name update
updated change log
Added change log
PROD-8149 - Pulled API changes
[PROD-8240] grunt
PROD-8378 - Grunt
PROD-8378 - Remove code which strip html tags so script tag can remove from draft also fixed jshint
PROD-8378 - sanitize topic title when added in draft
PROD-8378 -  sanitize topic title when add/edit
[PROD-8240] Fix moderations table UI issues in mobile screen
[PROD-8240] Fix Group and Activity tables UI issues in mobile screen
[PROD-8240] Fix error overflow outside container in smaller screens
[PROD-8332] Remove open class from message more option wrap once the dropdown is closed
[PROD-8310] Fix create new message uploader button UI issue
[PROD-8210] Set single profile avatars height to auto for better aspect ratio handling
Update: Grunt
Update: Grunt string-replace
Update: Version name update
...

# Conflicts:
#	src/bp-core/bp-core-actions.php
#	src/bp-core/bp-core-functions.php
#	src/class-buddypress.php
- Remove unwanted code
- Added header/footer template for readylaunch
- Fetch messages/notifications via ajax
- Folder structure updated
- Top ReadyLaunch Panel
- Bottom ReadyLaunchPanel
- Also modified code to convert array to register all menus
- Also update menu id and menu class for wp_nav_menu
- Called left sidebar template
- Added new file to load Top ReadyLaunch Panel menu in the left sidebar
- Store ReadyLaunch settings in the $this variable to make them reusable across different parts of the code.
- Register a new setting called Global Design Settings with options for Groups and Courses.
- Introduce a helper function to check if Groups and Courses are enabled for the left sidebar.
…instances, enabling better reuse and accessibility of methods inside the class.
- If setting is disable then 'My Groups' section not display
* release: (35 commits)
  Update: Grunt
  Update: Grunt string-replace
  Update: Version name update
  Added change log
  PROD-8401 - Use yoda condition
  PROD-8401 - fixed phpcbf
  PROD-8092 - Fix phpcbf
  PROD-8092 Update the conditions supporting the media theatre
  PROD-8331 Code refactor
  PROD-8331 Update filter name and argument
  [PROD-8401] Hide/show media actions buttons manually when ajax is off
  PROD-8331 Separate the email notification hook
  PROD-8308 - Fix phpcbf
  PROD-8308 fix issue for media and video download url
  PROD-8308 fix document download issue with group privacy
  PROD-8331 Update the filter argument
  PROD-8331 Update the filter arguments
  PROD-8331 Added filter to disable the notifications
  [PROD-7996] Updated BuddyBoss Platform French translation
  [PROD-8307] Fix profile avatar images dimension issue
  ...
- For courses, used filter to fetch courses
- bb_is_active_any_left_sidebar_section - In this function pass false and true which determine - if true, then check for return data otherwise it will check plugin active or not.
- Added condition $args['has_sidebar_data'] && $args['is_sidebar_enabled_for_courses'] ) to check below cases
- has_sidebar_data - Return data if its true otherwise its simple retrun integration so we can consider plugin is activate
- is_sidebar_enabled_for_courses - Return data if admin setting enable for courses in the left side bar
annrra and others added 30 commits January 29, 2025 22:58
- Fixed phpcs and phpcbf
…platform into feature-readylaunch

* 'feature-readylaunch' of github.com:buddyboss/buddyboss-platform:
  PRJ-55 - Style GIF no result UI
  PRJ-55 - Style GIF Picker
* PRJ-59: (60 commits)
PRJ-59 Show invite popup based on condition
PRJ-59 Update logic after conflict
PRJ-59 grunt
PRJ-58 - Added code for member dir
PRJ-55 - Style GIF no result UI
PRJ-55 - Style GIF Picker
PRJ-58 - Fixed jshint error - Fixed phpcs and phpcbf
[PRJ-59] Fix stylelint errors and import variables into members stylesheets
PRJ-58 - make common query function to avoid duplicate code - Fixed phpcs and phpcbf
PRJ-58 - Optimize JS code
PRJ-55 - Style Medium editor action form
PRJ-55 - Style different content formats
PRJ-55 -  Style mention and reposition medium editor toolbar
PRJ-55 - Move Medium Editor toolbar to correct position and style it
PRJ-55 - Style activity post form notice
PRJ-58 - Fixed phpcs
PRJ-58 - Fixed phpcbf
PRJ-55 - Remove self var as not need and fixed phpcs
PRJ-55 - Removed createThumbnailFromUrl function, make that function inside in readylaunch so it can utilise with other places
PRJ-55 - Fixed phpcbf - Optimise media/video/document code with updateMultiMediaOptions, updateMultiMediaToolbar, toggleMultiMediaOptions function. Created one common utility function to use that code - Fixed phpcs
...

# Conflicts:
#	src/bp-templates/bp-nouveau/readylaunch/css/members.css
…hange for send message, add localize strings for members js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants