Siva- Creating front-end for activity/description by pavanputti · Pull Request #4434 · OneCommunityGlobal/HighestGoodNetworkApp

pavanputti · 2025-11-21T21:09:44Z

Description

Related PRS (if any):

This frontend PR is related to the development backend PR. To test this frontend PR, you need to checkout the development backend PR.
…

Main changes explained:

Delete file A for removing unused components …
Update file B for including new pattern …
Create file C for introducing new components …
…

How to test:

Check out the current branch.
Run npm install and npm run start:local (or npm start) to run this PR locally.
Clear site data/cache.
Log in as an admin user.
Go to Community Portal → Activity (or open http://localhost:5173/communityportal/activity/ with an event ID from GET /api/events).
Confirm the Activity Agenda / summary page loads and shows event title, date, type, location, status, attendance, description, and schedule (and that loading/error states do not show “Something went wrong”).

Screenshots or videos of changes:

- Changed route from /communityportal/ActivityAgenda to /communityportal/activity/:activityid - Enables dynamic routing based on activity ID - Fixes 'Page Not Found' issue when accessing activity details

…rameter" This reverts commit 3231509.

- Changed route from /communityportal/ActivityAgenda to /communityportal/activity/:activityid - Enables dynamic routing based on activity ID - Fixes 'Page Not Found' issue when accessing activity details - Route now accepts activityid as URL parameter

netlify · 2025-11-21T21:10:04Z

✅ Deploy Preview for highestgoodnetwork-dev ready!

Name	Link
🔨 Latest commit	`e661f69`
🔍 Latest deploy log	https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69dabedab9842a0008e5d5bb
😎 Deploy Preview	https://deploy-preview-4434--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

- Added useParams import from react-router-dom - Extract activityid parameter from URL route - Added validation and error handling for missing activityId - Component now receives activityId from URL parameter - Prepared component for dynamic data fetching in next commit

- Added EVENT_BY_ID endpoint to URL.js for fetching single event - Implemented useState hooks for eventData, loading, and error states - Added useEffect to fetch event data when component mounts or activityid changes - Implemented fallback logic: tries EVENT_BY_ID endpoint first, falls back to fetching all events and filtering by ID - Added data transformation to map event model fields to component structure - Implemented loading, error, and no-data states with user-friendly messages - Added proper error handling and validation for API responses

- Enhanced data transformation with better date/time formatting helpers - Added event metadata display (date, type, location, status, attendance) - Improved schedule display with better formatting and resource location - Added image error handling with fallback to default image - Enhanced UI structure with organized sections for metadata, description, and schedule - Improved styling with conditional dark mode support for metadata sections - Better handling of missing or incomplete event data - Added formatted date display with full date string - Enhanced schedule items with background styling and better layout

- Enhanced error handling with specific messages for different error types (404, network, server, permission errors) - Added validation for activity ID (empty string check) - Improved date/time formatting with NaN validation to prevent invalid date displays - Enhanced schedule building with null/undefined resource filtering - Added type checking for numeric values (maxAttendees, currentAttendees) - Improved fallback API call error handling with network error detection - Better event data validation (object type check) - Enhanced error messages for better user experience - Fixed date display logic to handle null dates properly - Added array validation for schedule and resources data

…er functions to reduce fetchEventData complexity from 31 to ≤15 - Extract render functions to reduce ActivityAgenda complexity from 17 to ≤15

…l chaining

…urcesUsage import and ActivityAgenda route from both branches

- Fix dark mode by using CSS module classes (activityAgendaPage, activityAgendaDarkMode) for wrapper so theme variables apply - Move metadata, description, and schedule inline styles into ActivityAgenda.module.css - Add activityMetadata, metaRow, sectionHeading, descriptionBlock, scheduleItem classes with dark mode variants - Ensure section headings use var(--heading-color); schedule items use theme-aware backgrounds - Code cleanup: remove getPageClassName helper in favor of pageClassName; satisfy ESLint line length

Resolves ReferenceError 'getPageClassName is not defined' when loading or showing no-data state.

…agenda-dynamic-routing

sonarqubecloud · 2026-02-01T07:30:02Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

HemanthNidamanuru

Hi,

I tested this PR locally and the Activity Agenda page loads correctly. The event title, date, type, location, status, attendance, description, and schedule are all displayed as expected, and I did not see any error messages.

I noticed some misalignment in the elements on the page. Is the alignment also part of this PR?

Anusha-Gali

Hi Siva,

I have reviewed your PR locally and although it works as per requirement for the very first project id in both the modes, in all the other projects the image size is very large and is messing up the alignment of the text fields.

Hi @Anusha-Gali ,

Thanks for pointing this out. I’ll fix the image sizing for the other project IDs so it doesn’t affect the text alignment. I’ll update it in PR #4434: #4434
. Thanks!

- Constrain image column (max-width 380px) and image (max-height 320px, object-fit cover) - Add flex gap and align-items for consistent layout across events - Prevent large cover images from breaking text alignment in light/dark mode - Responsive: image max-height 280px on mobile, content full width

…ate literals - routes.jsx: Remove unused PRDashboardOverview, PRDashboardPromotionEligibility, PRDashboardTopReviewedPRs, PRDashboardDetails imports; remove commented PurchaseEquipment code - URL.js: Refactor TOP_CONVERTED and LEAST_CONVERTED to avoid nested template literals

…URL.js conflict Keep SonarQube-refactored TOP_CONVERTED/LEAST_CONVERTED (no nested template literals).

PRDashboardOverview, PRDashboardPromotionEligibility, PRDashboardTopReviewedPRs, and PRDashboardDetails components do not exist; use existing PRDashboard for /pr-dashboard/overview, promotion-eligibility, top-reviewed-prs, and details routes.

rohanrastogi311

Hi Siva,

Well done with this implementation.

Anusha-Gali

Hi Siva,

Thanks for fixing the issue. The UI is as per requirement.

…agenda-dynamic-routing # Conflicts: # src/routes.jsx

sonarqubecloud · 2026-04-11T21:37:33Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

one-community · 2026-04-12T04:30:26Z

Thank you all, merging!

pavanputti added 3 commits November 21, 2025 15:56

Update Activity Agenda route to support dynamic activityId parameter

3231509

- Changed route from /communityportal/ActivityAgenda to /communityportal/activity/:activityid - Enables dynamic routing based on activity ID - Fixes 'Page Not Found' issue when accessing activity details

Revert "Update Activity Agenda route to support dynamic activityId pa…

cc8a283

…rameter" This reverts commit 3231509.

pavanputti added DO NOT MERGE Don't merge PR do not review Do not review or look at code without full context labels Nov 21, 2025

pavanputti added 10 commits November 22, 2025 00:43

Refactor ActivityAgenda to reduce cognitive complexity - Extract help…

4474136

…er functions to reduce fetchEventData complexity from 31 to ≤15 - Extract render functions to reduce ActivityAgenda complexity from 17 to ≤15

Fix SonarQube issues: Replace isNaN with Number.isNaN and use optiona…

1adc2c7

…l chaining

Remove unused Resources import from routes.jsx

86f5f23

Merge origin/development: Resolve conflicts in routes.jsx - Keep Reso…

e3f7ad4

…urcesUsage import and ActivityAgenda route from both branches

Merge development into siva/activity-agenda-dynamic-routing

201ade3

pavanputti added Medium Priority and removed DO NOT MERGE Don't merge PR do not review Do not review or look at code without full context labels Feb 1, 2026

pavanputti added 2 commits January 31, 2026 23:24

Fix ActivityAgenda: use pageClassName in renderLoading and renderNoData

cd0b090

Resolves ReferenceError 'getPageClassName is not defined' when loading or showing no-data state.

Merge remote-tracking branch 'origin/development' into siva/activity-…

7685a19

…agenda-dynamic-routing

pavanputti changed the title ~~Siva/activity agenda dynamic routing~~ Siva/Creating front-end for activity/description Feb 1, 2026

pavanputti changed the title ~~Siva/Creating front-end for activity/description~~ Siva- Creating front-end for activity/description Feb 1, 2026

one-community added High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible and removed Medium Priority labels Mar 8, 2026

HemanthNidamanuru previously approved these changes Mar 10, 2026

View reviewed changes

Anusha-Gali suggested changes Mar 11, 2026

View reviewed changes

pavanputti dismissed HemanthNidamanuru’s stale review via a89eea9 March 15, 2026 01:47

pavanputti added 3 commits March 14, 2026 22:12

Merge development into siva/activity-agenda-dynamic-routing; resolve …

cc76211

…URL.js conflict Keep SonarQube-refactored TOP_CONVERTED/LEAST_CONVERTED (no nested template literals).

rohanrastogi311 previously approved these changes Mar 15, 2026

View reviewed changes

Anusha-Gali previously approved these changes Mar 17, 2026

View reviewed changes

Merge remote-tracking branch 'origin/development' into siva/activity-…

e661f69

…agenda-dynamic-routing # Conflicts: # src/routes.jsx

pavanputti dismissed stale reviews from Anusha-Gali and rohanrastogi311 via e661f69 April 11, 2026 21:36

one-community merged commit 06d5261 into development Apr 12, 2026
10 checks passed

one-community mentioned this pull request Apr 16, 2026

Frontend Release to Main [4.85] #5160

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Siva- Creating front-end for activity/description#4434

Siva- Creating front-end for activity/description#4434
one-community merged 20 commits intodevelopmentfrom
siva/activity-agenda-dynamic-routing

pavanputti commented Nov 21, 2025 •

edited

Loading

Uh oh!

netlify bot commented Nov 21, 2025 •

edited

Loading

Uh oh!

sonarqubecloud bot commented Feb 1, 2026

Uh oh!

HemanthNidamanuru left a comment

Uh oh!

Anusha-Gali left a comment •

edited by pavanputti

Loading

Uh oh!

rohanrastogi311 left a comment

Uh oh!

Anusha-Gali left a comment

Uh oh!

sonarqubecloud bot commented Apr 11, 2026

Uh oh!

one-community commented Apr 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

pavanputti commented Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related PRS (if any):

Main changes explained:

How to test:

Screenshots or videos of changes:

Uh oh!

netlify bot commented Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for highestgoodnetwork-dev ready!

Uh oh!

sonarqubecloud bot commented Feb 1, 2026

Quality Gate passed

Uh oh!

HemanthNidamanuru left a comment

Choose a reason for hiding this comment

Uh oh!

Anusha-Gali left a comment • edited by pavanputti Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rohanrastogi311 left a comment

Choose a reason for hiding this comment

Uh oh!

Anusha-Gali left a comment

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Apr 11, 2026

Quality Gate passed

Uh oh!

one-community commented Apr 12, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

pavanputti commented Nov 21, 2025 •

edited

Loading

netlify bot commented Nov 21, 2025 •

edited

Loading

Anusha-Gali left a comment •

edited by pavanputti

Loading