Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

Testimonial Delay #10147

Open
1 task done
dhruvabhat24 opened this issue Jan 16, 2024 · 13 comments
Open
1 task done

Testimonial Delay #10147

dhruvabhat24 opened this issue Jan 16, 2024 · 13 comments
Labels
🛠 goal: fix undefined 🔢 points: 3 undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)

Comments

@dhruvabhat24
Copy link
Member

dhruvabhat24 commented Jan 16, 2024

Has this bug been raised before?

  • I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?

Testimonial Section

Version of BioDrop (for example "v1.2.3")

v2.99.4

Description

I have received one testimonial a day before even that is closed #10144
But till now it is not displayed in my profile

Steps to Reproduce

edited by eddiejaoude

  1. create testimonial via issue
  2. testimonial not loaded until next deployment

Screenshots

No response

Do you want to work on this issue?

No

If "yes" to above, please explain how you would technically implement this

edited by eddiejaoude

  1. trigger deploy on new testimonial
  2. copy the curl command from the deployment GitHub Action to the Testimonial Action

run: curl -L -f https://biodrop.io/api/system/testimonials-forms?secret=${{ secrets.BIODROP_API_SECRET }}

@dhruvabhat24 dhruvabhat24 added 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) 🛠 goal: fix undefined labels Jan 16, 2024
Copy link
Contributor

To reduce notifications, issues are locked until they are 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) and to be assigned. You can learn more in our contributing guide https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md

@github-actions github-actions bot locked and limited conversation to collaborators Jan 16, 2024
@ChinmayMhatre
Copy link
Member

@dhruvabhat24 I believe it is visible now. But I'll open the repo for conversation to check if this delay is happening to more people. We can have a discussion on this issue is persists!

@EddieHubCommunity EddieHubCommunity unlocked this conversation Jan 20, 2024
@dhruvabhat24
Copy link
Member Author

dhruvabhat24 commented Jan 20, 2024

Yes now it is visible. I think it's better to make an announcement that if anyone is facing the same issue then they can discuss here

@eddiejaoude
Copy link
Member

eddiejaoude commented Jan 20, 2024

A new issue can be raised if it happens again, and we can then prioritise it again - but it seemed like it was a one off 🤞

ℹ️ eddiejaoude has some opened assigned issues: 🔧View assigned issues

@eddiejaoude eddiejaoude closed this as not planned Won't fix, can't repro, duplicate, stale Jan 20, 2024
@eddiejaoude eddiejaoude removed 🚦 status: awaiting triage Waiting for maintainers to verify (please do not start work on this yet) 💬 talk: discussion undefined labels Jan 20, 2024
@eddiejaoude eddiejaoude self-assigned this Jan 20, 2024
@dhruvabhat24
Copy link
Member Author

dhruvabhat24 commented Jan 24, 2024

@eddiejaoude I have received one more testimonial and facing the same issue as earlier. sorry for raising the same issue again and again.

@eddiejaoude
Copy link
Member

eddiejaoude commented Jan 24, 2024

No problem, but it has only been 1 hour since the testimonial - please can we give it time to see if the standard workflow works

ℹ️ eddiejaoude has some opened assigned issues: 🔧View assigned issues

@dhruvabhat24
Copy link
Member Author

@eddiejaoude okay

@dhruvabhat24
Copy link
Member Author

It's been more than 48 hours and it didn't fetch

@eddiejaoude
Copy link
Member

Yes, testimonials are reloaded when the app is deployed, you can see the code here https://github.com/EddieHubCommunity/BioDrop/blob/main/.github/workflows/vercel.yml#L55

From the deploy GitHub Action there has been no deployments for 3 days https://github.com/EddieHubCommunity/BioDrop/actions/workflows/vercel.yml

I think the code to load the testimonials should be moved from the deploy app to the testimonials GH Action

run: curl -L -f https://biodrop.io/api/system/testimonials-forms?secret=${{ secrets.BIODROP_API_SECRET }}

@eddiejaoude eddiejaoude reopened this Jan 27, 2024
@eddiejaoude
Copy link
Member

eddiejaoude commented Jan 27, 2024

I have manually reloaded testimonials, please check your account now

ℹ️ eddiejaoude has some opened assigned issues: 🔧View assigned issues

@dhruvabhat24
Copy link
Member Author

I have checked now but still it is displayed on account I think you need to make the testimonial sections to GitHub actions

@eddiejaoude eddiejaoude removed their assignment Feb 4, 2024
@eddiejaoude eddiejaoude added 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned) 🔢 points: 2 undefined labels Feb 4, 2024
Copy link
Contributor

github-actions bot commented Feb 4, 2024

The issue has been unlocked and is now ready for dev. If you would like to work on this issue, you can comment to have it assigned to you. You can learn more in our contributing guide https://github.com/EddieHubCommunity/BioDrop/blob/main/CONTRIBUTING.md

@eddiejaoude eddiejaoude self-assigned this Feb 4, 2024
@eddiejaoude eddiejaoude mentioned this issue Feb 4, 2024
1 task
@eddiejaoude eddiejaoude removed their assignment Feb 4, 2024
@eddiejaoude eddiejaoude added 🔢 points: 3 undefined and removed 🔢 points: 2 undefined labels Feb 4, 2024
@ArvindParekh
Copy link
Member

Hi! I'd like to help with this. Although I do not know about GH Actions (yet), I'm willing to learn on the go. I read about it and here's what I think needs to be done:

  • At present, the code to reload the testimonials lies in the vercel.yml file, where testimonials are only reloaded when the load-data job runs, which depends on the deploy workflow.
  • So I should figure out a way to update the testimonials without needing the deploy job to run.

Am I correct? Also, do I have to make changes to the testimonial.yml or just to vercel.yml?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🛠 goal: fix undefined 🔢 points: 3 undefined 🏁 status: ready for dev You can asked for this issue to be assigned (if not already assigned)
Projects
None yet
Development

No branches or pull requests

4 participants