Renatho De Carli Rosa

Results 87 comments of Renatho De Carli Rosa

> @donnapep, do you think it's fine to deploy it as it is and implement these other parts in other PRs? I think one issue would be fine for both...

> @renatho Could you add some additional context or screen shots of what these mean? It's a bit unclear to me. @donnapep Sure! I was referring to these parts (screenshot...

@m1r0 and @Imran92, since we had some iterations after the approval, I'll wait for a new approval, okay?

@Imran92, there's a cache, but after 1 or 2 minutes it's updated. Could you confirm if it works for you after 1 or 2 minutes? I tested a few times...

I just removed a not-needed conditional that I left accidentally: https://github.com/Automattic/wp-calypso/pull/95497/commits/456d3aa071b4c71b6057595fce550aca3bc1eb87. It wasn't being used at all since we are checking this in the parent component.

Sorry, but I did 2 last minute changes that I noticed while working on another PR: 1. I removed a not-needed conditional that I left accidentally: https://github.com/Automattic/wp-calypso/commit/456d3aa071b4c71b6057595fce550aca3bc1eb87. It wasn't being...

I just rebased to fix a conflict, so I'm doing a sanity test, and I'll wait for the tests and merge it.

I fixed most of the issues in the course/lesson/question editor with this PR: https://github.com/Automattic/sensei/pull/7646. The items not fixed that I also documented in the PR are: > - When navigating...

> I believe this is the same issue we have a user report for, and I see on my demo site. If you head to wp-admin> Appearance> Editor> Template>sensei-course-theme and...

Moved back to "To do" to keep it open for the other issues reported in the comment: https://github.com/Automattic/sensei/issues/7624#issuecomment-2261518634