Skip to content

Added freshdesk support #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anthonyoliai
Copy link

PR for #4 ; This adds a feedback widget on the bottom right in which users can give feedback to the website. Very preliminary as there are still things to consider such as styling. Right now the Loading app.. component is overriding with the feedback widget. How should we go about placing these apart? The free version also does not allow for customization of the color of the feedback button, etc. Right now i'm using a dummy freshdesk account, we can update it later to a TEC freshdesk account instead. Thoughts?

@vercel
Copy link

vercel bot commented Jan 6, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/gardens-swarm/tec-interface/narlx9xo7
✅ Preview: Failed

Copy link
Collaborator

@famole famole left a comment

Choose a reason for hiding this comment

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

Nice work @anthonyoliai, LGTM!

I have a small comment related to the place where the component is placed in the screen is overlapping the loading indicator.
Shouldn't we put it on the left? maybe @fioreb can give us her feedback.

@famole
Copy link
Collaborator

famole commented Jan 19, 2021

I've fixed the vercel configuration and redeployed this PR, this is the new link: https://tec-interface-git-feedbackwidget.gardens-swarm.vercel.app/#/ @fioreb

@fioreb
Copy link

fioreb commented Jan 20, 2021

Hey! Oh, this is really good!
Yes, I think it would be better on the left so we avoid that problem.

I know this will change eventually to have the TEC colours with the reskinning, but in the meantime can we change the colour to rgb(103 131 195)? That would make it more similar to the chromatic palette.

Thanks :D

@anthonyoliai
Copy link
Author

Hi! Thanks for the feedback. As far as I know freshdesk does not allow for change of colors, unless we pay. I agree with the positioning, moving it to the left seems like a good idea.

@BradleyC
Copy link

BradleyC commented Jul 8, 2021

This PR is getting dangerously stale... who can review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants