feelback-integrations icon indicating copy to clipboard operation
feelback-integrations copied to clipboard

Still active?

Open jfrome1 opened this issue 1 year ago • 13 comments

I'm very interested in using Feelback and installed it to my Astro Starlight site. The local form worked but getting data to the service didn't. I couldn't find a support email address. Is the project still active? If not, any suggestions for an alternate feedback system would be welcome. Thanks.

jfrome1 avatar Aug 31 '24 05:08 jfrome1

Yeah the project is active and under feature development.

What's your issue? Maybe it's a compatibility issue.

giuseppelt avatar Sep 02 '24 06:09 giuseppelt

Hi I'm also having issues with the tagged and split messages (message one works fine) returning 400 errors. Using Astro.

<FeelbackMessage 
contentSetId="a0d75f8f-e94b-4e67-ab72-8c595f37b914"
textAnswer="Thanks for your feedback"
/>
<FeelbackTaggedMessage 
contentSetId="c433c93f-53e2-495f-adb3-19244d2f3125"
 preset="like-dislike"
/>
<FeelbackSplitTaggedMessage
contentSetId="xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxxx"
    withMessage
    tags={{
        y: YES_TAGS,
        n: NO_TAGS
    }}
    titles={{
        y: "What did you like?",
        n: "What we should improve?",
    }}
/>

the error is - {"error":"invalid_param","message":"Invalid feelback value"}

JojokCreator avatar Sep 06 '24 01:09 JojokCreator

@JojokCreator That error is when something wrong is sent to the server (validation fail). So either the client in astro is sending something bad, or a configuration is not aligned.

Can you provide me the page (if it's public)? So I can check the error directly.

giuseppelt avatar Sep 11 '24 13:09 giuseppelt

https://stackblitz.com/edit/withastro-astro-d3vzyj?file=src%2Fpages%2Findex.astro <--- There's a minimal reproduction here

The feelbackMessage works but the Tagged and Split don't (both 400 errors). It could well be something I'm missing .

Also there's a typo here https://www.feelback.dev/docs/integrations/astro/

Should be import "astro-feelback/styles/feelback.css"; rather than astro-feedback right?

image

JojokCreator avatar Sep 11 '24 14:09 JojokCreator

I checked your snippet. Your using a content-set of type tagged-message with "happy", "neutral", "sad" values. You picked that preset 'feeling' when you created it.

So you need to match the same on your UI control.

<FeelbackTaggedMessage 
   contentSetId="xxxxxxxxxx"
   preset="feeling"
/>

And for the split-control you must use the same tag values. Follow the documentation for a stripe-like feedback system which uses the split control.

giuseppelt avatar Sep 11 '24 17:09 giuseppelt

Ah. I thought I might have been doing something wrong. It's working. Kind regards and a great product!

JojokCreator avatar Sep 12 '24 01:09 JojokCreator

I try to follow the guide for the mdx files but I do not see the feelback component when I go to the integration guide on the contentset and click go to documentation I get page not found error

Judimax avatar Sep 14 '24 12:09 Judimax

@Judimax if you are using MDX in Astro you should be able to just import the Feelback component and use it in your MDX like above + the import statement.

JojokCreator avatar Sep 15 '24 04:09 JojokCreator

@JojokCreator can you give an example snippet?. I might get the syntax wrong because I keep getting errors

windmillcode0 avatar Sep 16 '24 14:09 windmillcode0

@JojokCreator can you give an example snippet?. I might get the syntax wrong because I keep getting errors

https://stackblitz.com/edit/withastro-astro-d3vzyj?file=src%2Fpages%2Fposts%2F[...slug].astro,src%2Fpages%2Findex.astro,src%2Flayouts%2FLayout.astro,src%2Fcontent%2Fposts%2Ftest.mdx here you go

JojokCreator avatar Sep 16 '24 14:09 JojokCreator

in stackblitz I can only create an astro page how were you able to create a starlight astro page?

windmillcode0 avatar Sep 16 '24 15:09 windmillcode0

That's Astro but procedure with starlight would be the same.

JojokCreator avatar Sep 16 '24 15:09 JojokCreator

solved you can close the ticket

Judimax avatar Sep 16 '24 20:09 Judimax