Skip to content
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

Feat/update feedback component #282

Merged
merged 6 commits into from
May 16, 2024

Conversation

mariaozamiz
Copy link
Contributor

@mariaozamiz mariaozamiz commented May 7, 2024

📌 References

📝 Implementation

Replace old feedback-tool with feedback-component.

📹 Screenshots/Screen capture

Captura de pantalla 2024-05-07 a las 12 45 40

🔥 Testing

Launch the project and submit your feedback using the feedback component. To ensure the action was successful, navigate to the network tab. Once confirmed, head over to the ClickUp list and find there your submitted feedback.

src/app-config.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@p3rcypj p3rcypj left a comment

Choose a reason for hiding this comment

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

Thank you, @mariaozamiz, for adding the new component 🙌🏻 Everything looks good! There are just some minor things to be removed but everything is ok and the functionality works perfectly👌🏻

@mariaozamiz mariaozamiz requested a review from p3rcypj May 8, 2024 09:50
Copy link
Contributor

@p3rcypj p3rcypj left a comment

Choose a reason for hiding this comment

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

Everything looking good! Thanks!! @mariaozamiz

Copy link
Contributor

@MiquelAdell MiquelAdell left a comment

Choose a reason for hiding this comment

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

working fine, thanks!

@MiquelAdell MiquelAdell merged commit 4cafa9d into development May 16, 2024
@MiquelAdell MiquelAdell deleted the feat/update-feedback-component branch May 16, 2024 07:48
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.

3 participants