Skip to content

Sharadha taking over for Manvitha-Linkedin-Autoposter#3647

Merged
one-community merged 7 commits intodevelopmentfrom
Manvitha-Linkedin-AutoPoster
Apr 20, 2026
Merged

Sharadha taking over for Manvitha-Linkedin-Autoposter#3647
one-community merged 7 commits intodevelopmentfrom
Manvitha-Linkedin-AutoPoster

Conversation

@myeeli
Copy link
Copy Markdown
Contributor

@myeeli myeeli commented Jun 12, 2025

Description

image

https://docs.google.com/document/d/1raNHsAl2QMHm1w-3OmzersktXbl4_gyB865xL-y_NYQ/edit?tab=t.0

Related PRS (if any):

This frontend PR is related to this backend PR:
OneCommunityGlobal/HGNRest#1454

Main changes explained:

  • Implemented the LinkedIn auto poster feature where user can post anything on demand, schedule post , edit the scheduled posts and delete the scheduled posts
  • The data is not saved in the database
  • This is a temporary UI

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as owner user
  5. go to dashboard→ Send Emails

Screenshots or videos of changes:

https://drive.google.com/file/d/1s-P2BQlhnmWsLp90s8k2N0AaSi14ML66/view?usp=sharing

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify bot commented Jun 12, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 9bf70be
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e5de8cac18360008cc37bc
😎 Deploy Preview https://deploy-preview-3647--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.

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Sep 10, 2025
Copy link
Copy Markdown
Contributor

@akshith312 akshith312 left a comment

Choose a reason for hiding this comment

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

At first I encountered this error. I changed the connection strings. The back end ran fine.
But then I encountered an error while starting the front end.
image
image

Please let me know if I am doing any mistake while testing this.

@Stafvk
Copy link
Copy Markdown

Stafvk commented Sep 28, 2025

###PR review - 3647
I get the following error in my terminal
Screenshot (248)
Potential Fix:: There is already an exsisting property menubar in the same file which throws an error

The option is missing in the rendered UI
Screenshot (250)

@sanjeev29
Copy link
Copy Markdown
Contributor

Tested the PR changes. The tagged backend PR - 1454 was deleted. I'm testing this with the development branch on the backend. Getting a 404 error when I click on Post Now.
Screenshot 2025-10-17 at 5 43 33 PM

Copy link
Copy Markdown
Contributor

@sumedhkumar96 sumedhkumar96 left a comment

Choose a reason for hiding this comment

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

Tried to test this by running the frontend and backend in my local but the LinkedIn post fails with a 404. I see that the URL localhost:4500 is hardcoded in the code and that's the reason for 404. Please provide complete instructions to test this.

Screenshot 2025-10-18 181219

@apoorvajainrp21
Copy link
Copy Markdown
Contributor

I am getting the following error:
Screenshot 2025-10-19 214433

@tomkkl
Copy link
Copy Markdown

tomkkl commented Oct 26, 2025

Hi Manvitha,

Testing this does not take me to the /announcements and it makes me stuck at /dashboard. I have attached an image down below. Hope this helps!
Screenshot 2025-10-25 at 6 18 22 PM

@Vinay944924
Copy link
Copy Markdown
Contributor

Hi Manvitha,

I tried to post a blog in linkedin but it was throwing an error and I have provided the screenshot below. Please let me know if I am doing anything wrong.

Screenshot 2025-11-01 at 1 24 20 PM

@ManeeshBuddha21
Copy link
Copy Markdown

Tested locally. The LinkedIn Post Editor loaded fine under Dashboard -> Send Emails.
Able to create and schedule posts from the UI, and the layout looks clean.
Tried posting sample text and an image , UI responded correctly but returned “Failed to post to LinkedIn”
21

@Sriamshreddy000
Copy link
Copy Markdown
Contributor

Hi Manvita,

I tested this PR along with the related backend PR. The LinkedIn Post Editor UI loads correctly and allows text entry, but posting fails with the error message “Failed to post to LinkedIn.”

Attaching a screenshot for reference.
Screenshot 2025-11-11 at 12 36 41 PM

Copy link
Copy Markdown

@debadyuti23 debadyuti23 left a comment

Choose a reason for hiding this comment

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

Hi @myeeli I have tested the PR with the instructions you provided in this PR and for the auth tokens in backend PR, however it is resulting unauthorized as shown in the video.
I have done multiple refreshes and installation resets still the issue persists. I even have tested the linkedin api with same creds, it is working fine.
Additionally, I have encountered this issue in backend, make sure your feature is synced with recent changes.
PR_3647_backend_error_log
Therefore please help me understand if I am missing something in testing.
Another thing I noticed that backend api localhost:4500 is hardcoded, it should be changed

@SwathiAngadi
Copy link
Copy Markdown
Contributor

Hi Manvitha,

Tried to review your PR. Not able to post on LinkedIn. Getting error. Looks like URL is hardcoded with localhost:4500. And also UI looks broken in Dark Mode.

Screenshot 2025-12-09 172352 Screenshot 2025-12-09 172421

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 Manvitha,

I tried reviewing your PR locally and getting the same issue as other reviewers. Kindly look into it.
Even in dark mode we cannot see the text we have typed and the date or time selected.
Screenshot 2026-01-06 at 10 49 50 PM
Screenshot 2026-01-06 at 10 51 35 PM

Copy link
Copy Markdown

@Vikas-8055 Vikas-8055 left a comment

Choose a reason for hiding this comment

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

Hi Manvitha, I tested this PR along with backend PR #1454. The LinkedIn Post Editor UI loads correctly and text entry works, but clicking "Post Now" fails with the error "Failed to post to LinkedIn".

512925691-ff1542aa-bdc7-40be-ae7f-8676d414bb22

Copy link
Copy Markdown

@Shravan-neelamsetty Shravan-neelamsetty left a comment

Choose a reason for hiding this comment

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

Hi @myeeli, I have tested this PR locally by checking out the Manvitha-Linkedin-AutoPoster branch and navigating to /announcements. The LinkedIn Post Editor UI loads correctly and displays all expected components including the text editor, upload media button, date/time picker, "Post Now" button, and "Scheduled Posts" section. However, I immediately encountered the critical error "Failed to post to LinkedIn" displayed in the notification at the top right of the page.
3647

Copy link
Copy Markdown

@Ganesh112001 Ganesh112001 left a comment

Choose a reason for hiding this comment

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

Tested PR #3647 but cannot access the Announcements page—all navigation attempts redirect to Dashboard even when logged in as Owner. The route appears inaccessible or not properly configured in this branch.

Screenshot 2026-01-23 at 2 18 41 AM

@maithili20
Copy link
Copy Markdown
Contributor

maithili20 commented Jan 31, 2026

Hi @myeeli
I tried reviewing your PR. When I try to post, it throws the error shown in the screenshot
Screenshot 2026-01-31 at 5 22 23 PM
Screenshot 2026-01-31 at 4 14 27 PM
I can post via postman to my personal profile though. s the intended behavior to post to a company profile or a personal profile?
Screenshot 2026-01-31 at 5 30 15 PM
The text and date fields aren’t visible in dark mode
Screenshot 2026-01-31 at 5 14 54 PM
Screenshot 2026-01-31 at 5 08 52 PM

Comment thread src/components/Announcements/index.jsx Outdated
Comment thread src/components/Announcements/index.jsx Outdated
Comment thread src/components/Announcements/index.jsx Outdated
Comment thread src/components/Announcements/index.jsx Outdated
@one-community one-community changed the title Manvitha-Linkedin-Autoposter Sharadha taking over for Manvitha-Linkedin-Autoposter Apr 6, 2026
Copilot AI and others added 2 commits April 20, 2026 05:29
… ENDPOINTS, restore email/title props, fix duplicate menubar key

Agent-Logs-Url: https://github.com/OneCommunityGlobal/HighestGoodNetworkApp/sessions/bb774a64-319d-44f1-9c15-27b1ee86403b

Co-authored-by: SharadhaKasiviswanathan <74849358+SharadhaKasiviswanathan@users.noreply.github.com>
…ilList state

Agent-Logs-Url: https://github.com/OneCommunityGlobal/HighestGoodNetworkApp/sessions/bb774a64-319d-44f1-9c15-27b1ee86403b

Co-authored-by: SharadhaKasiviswanathan <74849358+SharadhaKasiviswanathan@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

@copilot resolve the merge conflicts in this pull request

Merge conflicts were already resolved in the previous commit 84f23f0. The branch is currently up to date with no unresolved conflicts.

…heck

# Conflicts:
#	package-lock.json
#	src/components/Announcements/index.jsx
#	src/utils/URL.js
@SharadhaKasiviswanathan SharadhaKasiviswanathan dismissed stale reviews from ManojPuttaswamy, naznin07, Ganesh112001, Anusha-Gali, debadyuti23, and sumedhkumar96 April 20, 2026 07:16

Superseded by the latest branch state: the referenced review items are addressed on head 8e507d4, all review threads are resolved, and the current GitHub Actions/Sonar checks are green.

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit cd25657 into development Apr 20, 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.