Antonis Lilis
Antonis Lilis
I've noticed a minor UI bug that occurs when the reader refreshes while there is cached content displayed. In this occasion the "Fetching posts" overlaps with the content as seen...
Makes sense @thomashorta 👍 > We were aware of the problem but were not sure about the exact scenario and it looks like we don't have a GH issue I...
Awesome work team 🏅 I had a quick look at the parts that involve the Dynamic Dashboard cards and did not detect any issues. I also tested by creating cards...
> I would expect a top card to be just below the menu (like in pc8HXX-1sr-p2) and is below the blogging prompts. I drafted a fix for this in https://github.com/wordpress-mobile/WordPress-Android/pull/20512...
Linked up the two related crash reports above. The issue still persists in the latest versions of the app.
> @antonis Pr is ready for review! Thank you for your work on this @07jasjeet 🙇 My plan is to review this in detail next week. I was wondering if...
> Alright, so each commit will have its own PRs. The above PR split looks good to me 👍
Hey @07jasjeet 👋 As discussed on Slack I'll dive deeper on the PR changes and provide feedback on this PR next week before deciding on if/how to split your contribution...
> Not sure on how to solve the Specific imports should be used for FluxC Store inner classes check style issue. This was being used before in older implementations without...
Hey @07jasjeet 👋 thank you for the submitted fixes. All the checkstyle issues have been resolved and most of the Detekt issue. The following Detekt issues persist. Let me know...