πŸ‡§πŸ‡·Brazil @ashley_herodev

Account created on 10 August 2023, over 1 year ago
  • Senior Software Engineer at HeroDevsΒ 
#

Recent comments

πŸ‡§πŸ‡·Brazil ashley_herodev

The patch failed to apply due to trailing whitespace, but since I didn't find any whitespace locally I suspected of an EOL issue. So I just converted CR LF to LF

πŸ‡§πŸ‡·Brazil ashley_herodev

Fixing the patch in a few moments

πŸ‡§πŸ‡·Brazil ashley_herodev

Thanks @Gauravvvv for applying the interdiff file 47-55 and recreate the patch for me

πŸ‡§πŸ‡·Brazil ashley_herodev

Added screenshots before and after and addressed items on #53 comment

πŸ‡§πŸ‡·Brazil ashley_herodev

I've just start looking into it, I should be able to provide some feedback to comment #53 soon

πŸ‡§πŸ‡·Brazil ashley_herodev

Hiding old patch for clarity

πŸ‡§πŸ‡·Brazil ashley_herodev

Ops, my IDE messed with Drupal's code standards, I will try to fix it

πŸ‡§πŸ‡·Brazil ashley_herodev

ashley_herodev β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡·Brazil ashley_herodev

Thanks at @WalkingDexter, I have added a test-only patch which should fail to prove the test coverage for #27

Production build 0.71.5 2024