Balneário Camboriú
Account created on 30 September 2019, almost 6 years ago
#

Merge Requests

More

Recent comments

🇧🇷Brazil julio_retkwa Balneário Camboriú

I've tried all the steps under the https://project.pages.drupalcode.org/la_eu/setup/ (including the short version)

And I was not able to reproduce, there is anything specific on the setup that may is resulting that issue? e.g
This drush updb was executed inside ddev setup?
The setup was using the same php version as settled on ddev php_version: "8.3" ?
It was an initial setup or it happened with an already existed DB?

any other information is useful for this investigation

Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

I'm taking a look on this

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @joachim I see you worked on the branch '3530764-options-in-the'
is that something you taking care atm? do I got ahead of myself and you are already dealing with that on this branch?

Let me know if That makes sense or if I thought it was too simple
Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @joachim

Let me know if that makes sense? Sub-theme I guess?

Thanks!

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

I'm sorry but I'll need to give up this one, I'm out of ideas on how to solve the test problems that I'm facing

🇧🇷Brazil julio_retkwa Balneário Camboriú

anyone can review it?

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hey all
I've added a test scenario let me know if that makes sense
any reviews are welcome :D https://git.drupalcode.org/project/drupal/-/merge_requests/12390

🇧🇷Brazil julio_retkwa Balneário Camboriú

I created a MR against branch 8.x-2.x-dev but its failing on composer step
https://git.drupalcode.org/issue/google_calendar_service-3528488/-/jobs/...

I'm missing something here?

PS: I see there are some PHPCS issues on branch 1.x should those be addressed as well? Let me know
https://git.drupalcode.org/issue/google_calendar_service-3528488/-/jobs/...

Thanks!

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @danrod sorry, yes

I was not aware that you was working on this, sorry for jumping in

🇧🇷Brazil julio_retkwa Balneário Camboriú

Anyone can review this please? Thanks

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Does this happens in any other module/theme or just on gin?

🇧🇷Brazil julio_retkwa Balneário Camboriú

I've updated MR based on feedback, let me know if that makes sense

🇧🇷Brazil julio_retkwa Balneário Camboriú

Can anyone review this please?

🇧🇷Brazil julio_retkwa Balneário Camboriú

Anyone can review this? If needs any correction let me know

🇧🇷Brazil julio_retkwa Balneário Camboriú

Can anyone review this and check if seems covered right?

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @rfmarcelino can you review this one as well ? Thanks

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3525113-europe-timezone-past to active.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3525113-europe-timezone-past to hidden.

🇧🇷Brazil julio_retkwa Balneário Camboriú

My go to:

- Will Huggins (zoocha-will)
- Nico Grienauer (grienauer)

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @rfmarcelino can you review this piece ? Thanks

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @smustgrave I've rebased to last 11.x version

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @adinancenci I've just tested this following the steps
1. Upload an image with an uppercase extension.
2. Try to open the fallback.

And is working fine

Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

I've added a MR but I'm not quite sure if that sounds reasonable

I appreciate any inputs here

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @mxh I've added your suggestion to MR

Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @drupalmatts while testing this I realized that tag 3.1.0 has already the changes of this commit

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 11.x to hidden.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3515604-use-drupal-behaviours to active.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @sandip thank for the suggestion! - I've added that on my last commit

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @aaron.ferris I've just followed the test instructions and seems working fine

Thank you!

video proof

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

I created an MR for this, but I'm not sure about regression tests since I cannot set IE on my actual setup. I've checked the elements styles on chrome 135.0.7049.85 and looks same as before. If anyone could provide some default test instructions (if there is any guideline for manual testing on core), that would be great.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3519513-remove-remaining-ie11 to active.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3519513-remove-remaining-ie11 to hidden.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa changed the visibility of the branch 3519513-remove-remaining-ie11 to hidden.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Okay let me know when and if we will proceed at time
thanks

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @mstrelan I've just add initial commit for that but I'm not sure about the summary, I'll update that but let me know if something seems off

Thanks!

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @kim.pepper I've added a initial commit to this issue but I'm not sure if we are missing test instructions or may we need add new test cases for it ?

As far I understood it is just a sub task from #833734

Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @aaron May you can review it for me?

would this require any test instructions ?

Thank you

🇧🇷Brazil julio_retkwa Balneário Camboriú

#16 Hi @bnjmnm Thanks for jumping into that, I've changed summary wrongly.
Here is the result of my MR:

🇧🇷Brazil julio_retkwa Balneário Camboriú

I'll start this one, thanks!

🇧🇷Brazil julio_retkwa Balneário Camboriú

julio_retkwa made their first commit to this issue’s fork.

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @smustgrave I've just updated, but I'm not sure if following are right:
* Release notes snippet
* Introduced terminology
* Remaining tasks

May you can assit me with that?

Thank you!

🇧🇷Brazil julio_retkwa Balneário Camboriú

#14 Hi @smustgrave sorry I'm not sure if I get it right. I should add some test coverage on which stage exactly? (I presume nightwatch?) and the CR would stand for Change Request on contrib themes yes?

🇧🇷Brazil julio_retkwa Balneário Camboriú

Hi @bnjmnm thanks for the insights I've updated my MR based on your suggestion

off topic here:
My previous commits where made based on the following issue:

There is any issue related to that already opened? it basically happens on any media library item that has a text that goes less than 80% of wrapper size

Production build 0.71.5 2024