-
Notifications
You must be signed in to change notification settings - Fork 144
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
Implement new loading everywhere #855
Comments
hello @fricoben please kindly assign :) i would love to take on this issue and get it done |
hey man @fricoben can i work on this |
@fricoben can i be assigned this task |
@fricoben kindly assign me this task |
@fricoben Please can I be assigned this issue? Will be sure to deliver a good job |
@fricoben Please i am willing and ready to work on this. |
Hi @fricoben , could you please assign this issue to me? I’ll make sure to deliver quality work. |
Hello @fricoben , can i work on this? |
Hey, there @fricoben I have checked the wrapper component, and once it's done I can take this task. I understand the requirement and it will be better to have an HOC that will streamline the process of displaying loading UI through the application and with consistent styles. If we need to I can also work on the wrapper component. |
Hello @fricoben can I work on this. I would love the opportunity to solve this issue |
@fricoben Please can I be assigned this issue? I will be sure to deliver quality work and I understand the issue better from the previous one just merged. |
Hi @fricoben , please can I be assigned this issue? |
@fricoben |
Hey! @fricoben Mind if I give this one a try? I've already reviewed the new reusable loading component and am ready to apply it to the app. Let me know if that works for you! 😊 |
Hi @fricoben could I work on this? I believe it should be completed in about a few hours. Thanks |
Hello @fricoben. I'm Wheval, a Frontend developer with solid experience in building clean and functional user interfaces. ETA is 3 days |
Hello, @fricoben, can I work on this issue? Due to the size of the project, I will estimate 3 days to fix the issue |
Hi @fricoben, I will run a search across the entire codebase for the current customized skeleton new standard loading wrapper, and replace them all with the loading wrapper created in the mentioned issue. I will then make sure that all UI works in the pages as expected, and remove all unused imports of the skeleton wrapper ETA: 3 days |
@fricoben |
Hello @fricoben My approach to tackle this issue: I wll locate all instances of the current customized skeleton components and will replace them with the new standardized loading wrapper to maintain consistency across the UI. ETA-3days |
@fricoben is this issue still needed? please can I be assigned this issue? Will be sure to deliver a good job |
@fricoben I'd like to handle this task |
Can I take this issues? I'm a frontend and smart contract developer and a first time contributor in this repo. |
This is not labeled first-time contribution. I'll love to implement and get it done. |
Hey Fricoben, I am a returning contributor to OnlyDust. I've already worked on several projects here, notably kakarot and starknet.js, where I successfully completed multiple issues. Of course, I'll make sure to follow all the community rules and contributing guidelines carefully, as I've done before. I have decent experience with NextJS and TypeScript, you can count on me to handle the updates correctly across the codebase without lot of back and forth. ETA: 2d |
Hi, @fricoben, I am a new contributor and I would like to take on this issue. |
Hello Sir @fricoben , |
Description
This issue is linked to #853 once this is done we'll need to populate the change to the whole codebase by:
This will need some time and patience, be rigorous and you'll be rewarded! If I have to test again all skeletons when I review go back and forth ... the reward will be smaller.
I'll wait for @Ugo-X to finish :)
The text was updated successfully, but these errors were encountered: