Skip to content

Siva -Fix activity FAQ/Comments route#4553

Merged
one-community merged 15 commits intodevelopmentfrom
siva/activity-faq-comments-section
Mar 23, 2026
Merged

Siva -Fix activity FAQ/Comments route#4553
one-community merged 15 commits intodevelopmentfrom
siva/activity-faq-comments-section

Conversation

@pavanputti
Copy link
Copy Markdown
Contributor

@pavanputti pavanputti commented Dec 13, 2025

Description

Screenshot 2026-01-31 at 7 32 28 PM Screenshot 2026-01-31 at 9 54 59 PM

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:

  • routes.jsx: Add route for /communityportal/activity/:activityid/comments (and /Comments) rendering full Activity with FAQs tab; add base route /communityportal/activity/:activityId; remove unused Resources import.
  • Activity.jsx: Support initialTab, render ActivityFAQs in FAQs tab; add event type line, Location/Link labels, status pill, participant avatars.
  • Activity.css: New global stylesheet (replaces Activity.module.css) so header, event card, tabs, and calendar are styled.
  • ActivityFAQs.jsx: Change search placeholder to "Search".

How to test:

  1. Check out siva/activity-faq-comments-section, run npm install and npm start.
  2. Log in with Community Portal access.
  3. Open /communityportal/activity/1/comments.
  4. Confirm full activity page loads with FAQs tab active, styled header (event card, tabs, calendar), and FAQ section below.
  5. Confirm in dark mode if needed.

Screenshots or videos of changes:

Before:
Screenshot 2026-01-31 at 9 52 40 PM

After:

Screen.Recording.2026-01-31.at.9.50.08.PM.mov

@pavanputti pavanputti added the do not review Do not review or look at code without full context label Dec 13, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 13, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 066e7e6
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69b6084e1df3850008f88e7d
😎 Deploy Preview https://deploy-preview-4553--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@pavanputti pavanputti added Medium Priority and removed do not review Do not review or look at code without full context labels Feb 1, 2026
@pavanputti pavanputti changed the title Add TODO comment for activity FAQ/Comments route Siva -Fix activity FAQ/Comments route Feb 1, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 1, 2026

@one-community 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
Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

I tested this PR locally and the activity page loads correctly with the FAQs tab and section working as expected. Everything looks good.

Image Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Siva,

I have reviewed your PR locally and through the functionality does work as expected. Comparing it to the reference image - the Frequently asked questions section above the dropdown is not centered, and event the title is too close to the border.
Issue
Image

Working Screenshots
Image
Image
Image
Image

@sonarqubecloud
Copy link
Copy Markdown

@pavanputti
Copy link
Copy Markdown
Contributor Author

Hi Siva,

I have reviewed your PR locally and through the functionality does work as expected. Comparing it to the reference image - the Frequently asked questions section above the dropdown is not centered, and event the title is too close to the border.

Hi @Anusha-Gali,

Thank you for reviewing the PR and pointing this out. I’ve updated the Frequently Asked Questions section to center it properly and adjusted the title spacing from the border to better match the reference image.

Could you please check the latest changes in PR #4553 (#4553
) when you get a chance? Thanks! 🙂

Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Siva,

Well done with this implementation.

Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Siva,

Thanks for fixing the issue, i have re-reviewed and the UI is almost similar to the requirement.

Image Image

Copy link
Copy Markdown

@Vishnu10-hub Vishnu10-hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

Image Image

I have tested this PR locally. The FAQ section loads correctly, the event header, tabs, and calendar are displayed as expected. Functionality works properly in both light and dark modes. Everything appears aligned with the requirements

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 7281c43 into development Mar 23, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants