Jayant Bhawal
Jayant Bhawal
Hey apologies I missed the notification. Sure, feel free to take this up, @sidtohan!
Because this will be a visual change, please share your approach/idea for changes before implementing anything.
Something different is acceptable, @Ankur1493 and @sidtohan The screenshot shared belongs to the landing page. This issue talks about adding a CTA to the docker app.
Sure, @dominicghizzoni
@larrikin-coder, there's an existing PR for it, #567 You may decide to continue on that (or look at the review already done on that) as a starting point.
Yes @joshh35
Thanks for your contribution, @vishalmishraa! Based on the screenshots it's certainly a decent improvement. I would love to accept this PR, but it needs a few improvements to avoid performance...
Linter failed. We need to sort out the linting issues but for now you can check our contributing guide to see how to fix that. Let us know if it...
Directionally I think yes. Your last commit looks better too. I haven't done a proper review but I'll ask the team to check it out.
This PR could use some tests. Especially with various kinds of logs that might come through.