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
Reusable FAQ Component #1287
Reusable FAQ Component #1287
Conversation
Thanks @Monilprajapati @ayush0033 can you check if this is useful together with @iiAnna? |
Hey @ayush0033 ! |
Hey @ayush0033 Please take a look and take a futher steps |
Hey @frikky Please take a look! and let me know what should i need to do further |
Hey @Monilprajapati, share the screenshot or screen recording of the view so we can get a better idea. |
Okkk @ayush0033 Shuffle_FAQ.mp4 |
Hello @Monilprajapati, thank you for your work on the FAQ component. However, we've already implemented a similar feature on the front page. Your efforts are truly appreciated, and I'll keep your FAQ in mind for future needs. Thank you once again for your dedication and hard work. Here is our existing FAQ component: Shuffle.-.admin.mp4 |
My pleasure, Yeah that i know but i have raised this issue to change the FAQ section on the pricing page so you can change the UI their, shuffle_faq.mp4 |
@ayush0033 One more thing I'd like to mention is that Shuffle needs to revamp its code. There's an excessive amount of code in a single file, and React isn't being utilized efficiently. The same code is repeated throughout. Please take a look into this, and if you find that it requires attention, I'm in :) |
I will definitely take a closer look at this for the pricing page to ensure its reliability. |
Heyo! Just to chime in:
What you are talking about repeated code tho - that's indeed where components shine, and we do that in most of the code written the last few years. For an FAQ section, this is overdramatization of a very simple problem where it in reality doesn't matter much. Either way: @ayush0033 will look into it all again and we'll figure something out :) |
Hey @frikky I wanted to take a moment to apologize for any unintended tone in my previous message regarding the code structure. I realized that my wording may have come across as more critical than intended 🥲 🥲 Please know that my intention was solely to offer constructive feedback aimed at improving the efficiency and organization of the codebase. I deeply value the contributions of everyone involved in the project, including yourself, and I never meant to undermine that 🥲 I appreciate your patience and understanding, and I'm fully committed to working together to find the best solutions for Shuffle. Let's continue collaborating positively to enhance the project 🙂 I will provide you with the codes that I was referring to, along with proposed solutions for each issue. I will then proceed to solve them one by one by creating individual issues. Just give me some time :) |
So here, I have found 7 things that I was referring to in 2 days, along with their solutions. I have attached the video in which I have shown everything to get the idea quickly.1. There is one component called
|
Hey @frikky, please check this out. I am very excited to work on it :) |
Hey @frikky @ayush0033 :) |
Heyo! Sorry, we're busy building stuff, and don't pay enough attention here. Thanks for this - it could help a lot! Are you available for a call? I think we should have a discussion about this and your contributions, as you are bringing up some good points, along with some I disagree with. Answering your message elsewhere so we can get it set up ;) |
As per Issue #1273 , I have edited the FAQ component and improved its user interface. You can add this component anywhere, as I have raised an issue on the pricing page FAQ section, where it needs to be applied first.
@frikky Check this out!
I am currently addressing Issue #1274. Please assign it to me.