🇺🇸United States @baldwinlouie

@vinodhini.e and @divya.sejekan

I'm sorry I didn't attach steps to reproduce. The actual font dropdown works and saves. However, when you go back to the site, the font does not change.

I added a steps to reproduce section.

HTH

@yas Please review this patch. I've opened another issue for the CI/CD failures.

baldwinlouie → created an issue.

baldwinlouie → created an issue.

Looks good to me. @yas, moving to RTBC

@yas, Please review this initial Recipe check-in. I will create a CFn after this is commited to do additional testing.

baldwinlouie → created an issue.

@transmitter please review this patch if you have a chance. Let me know if it fixes the issue.

@yas, please review and merge when you can.

baldwinlouie → made their first commit to this issue’s fork.

@transmitter, Currently the light config options do not seem to work. I will look at implementing a Light version.

@yas, I am closing this issue. This issue went away after updating from Drupal 11.1.8 to 11.2.0

baldwinlouie → created an issue.

@yas, please review this patch. This just adds a description to the yaml file.

baldwinlouie → created an issue.

@yas Please review this patch. This is the initial version of composer.json. Now I can test composer require drupal/cloud_orchestrator_recipe

baldwinlouie → created an issue.

@yas, please review this patch. This removes the older dashboard.

baldwinlouie → created an issue.

@yas, please review this MR. This moves the drupal/cloud_orchestrator tabbed dashboard into the cloud_dashboard module.

baldwinlouie → created an issue.

@yas, thank you for the patch. It looks good to me. Changing to RTBC.

@yas, please review this patch. This makes a second adjustment to this issue: https://www.drupal.org/project/cloud/issues/3517479 📌 Update SPA to pass scope parameter during authentication Active

It also fixes a Login error with SPA. I needed to recompile the SPA to get rid of a debugger statement.

baldwinlouie → created an issue.

@yas, please review this patch. This fixes the following:

• Updating AMIs to Ubuntu 24.04LTS.
• Updating UserData to fix pip install for cfnsignal
• Removing need to explicitly declare PHP 8.3. PHP 8.3 is the default version for Ubuntu 24.04LTS
• Adding assert.active = Off to Dockerfile. assert.active is deprecated and can return a PHP warning. This shows up during a Drupal cache clear.

baldwinlouie → created an issue.

@yas, please review this patch. This adds the last piece to making the SPA work with SimpleOauth:6.x .

baldwinlouie → created an issue.

@yas, Please review this patch. This add support for creating the Scope entity for drupal/simple_oauth:6.x

baldwinlouie → created an issue.

@yas, please review this patch. It fixes some Drupal 11 compatibility issues.

baldwinlouie → created an issue.

@esha_kundu Thank you for the patch. It looks good to me. Marking RTBC

@esha_kundu, @rohitsharma401 Thank you for the patch. It looks good to me.

@yas, please review this patch.

baldwinlouie → created an issue.

@yas Please review this patch. This fixes the 500 error from our config forms.

@yas, I fixed the ESLint errors. Please review when you get a chance.

baldwinlouie → created an issue.

@yas, please review this patch. This fixes the ordering issue with dark-theme.css. I'm forcing it to be rendered after all the bootstrap libraries are declared.

Also, I fixed the issue with jquery.once. In global.js, there were still references to jquery.once which has been deprecated for drupal/once. This code was causing Javascript errors and the "tabs" on the dashboard were not working.

baldwinlouie → created an issue.

@jaydeep_patel, thank you for the patch. It looks good to me. Moving to RTBC

@jaydeep_patel, thank you for the patch. It looks good to me. Moving to RTBC

@yas. Please review this patch. This updates the maclof/kubernetes-client to 0.31 in prep for Drupal 11

baldwinlouie → created an issue.

@yas, I updated the MR. It looks like it is mergeable now.

@yas, please review this patch. This removes the deprecated default_argument_skip_url from our view configuration files.

This also removes the {% apply spaceless %} twig filter, which has also been deprecated.

baldwinlouie → created an issue.

@yas, please review this patch. This adds support for Drupal 11 into the submodules.

baldwinlouie → created an issue.

@sayan_k_dutta , thank you for the patch. It looks good to me now. Moving to RTBC.

@sdhruvi5142, Thank you for the patch. I have one comment. Can you please check it out.

@sayan_k_dutta, Thank you for the patch. I have one comment. Can you please check it out.

@alok_singh, Thank you for the patch. I have one comment. Can you please check it out.

@kanchan bhogade, Thank you for the patch. I have one comment. Can you please check it out.

@alok_singh, Thank you for the patch. I have one comment. Can you please check it out.

@dhruv.mittal, Thank you for updating the patch. It looks good to me now. Marking RTBC.

@lavanyatalwar and @jaydeep_patel,

Thank you for the patch. It looks good to me.

@dhruv.mittal Thank you for supplying this patch. It looks good to me. Moving to RTBC.

@jaydeep_patel. Thank you so much for your patch. The compiled main.css looks to have a lot of change. I think most of them is because the npx prettier did not run. Can you please review the https://git.drupalcode.org/project/rigel/-/blob/8.x/README.md?ref_type=h... and run grunt to do the SCSS compilation. It will run npx prettier so the main.css is formatted properly and we can review the changes made to main.css

@jaydeep_patel. Thank you so much for your patch. The compiled main.css looks to have a lot of change. I think most of them is because the npx prettier did not run. Can you please review the https://git.drupalcode.org/project/rigel/-/blob/8.x/README.md?ref_type=h... and run grunt to do the SCSS compilation. It will run npx prettier so the main.css is formatted properly and we can review the changes made to main.css

@dhruv.mittal Thank you for the patch. Can you please double check the compiling of the SCSS? The changes in _component.css is fairly minimal, but main.css has a lot of changes to it.

@jaydeep_patel and @dhruv.mittal, Thank you for the patch. I have made one comment on the code. Can you please review it.

@dhruv.mittal and @jaydeep_patel , Thank you for the merge request. Can you please double check the compiling of the SCSS? The changes in _component.css is fairly minimal, but main.css has a lot of changes to it.

@dhruv.mittal, I tested this patch and it looks good to me. Setting to RTBC

@dhruv.mittal

Thanks for looking at this ticket. I think `sass` is the option we go with.

@dhruv.mittal, thank you for the patch. It looks good to me. Changing to RTBC

@anish.ir, Thank you for providing the patch. It looks good to me. Moving it to RTBC.

yas → credited baldwinlouie → .

@yas, I've updated the patch to fix the stylelint issues.

@yas, please review these patches.

baldwinlouie → created an issue.

@yas, I'll create the 7.x and 8.x branch

@sachin.addweb. Thank you for the fixes to the patch. It looks good to me now.

@sachin.addweb, Thank you for the updated patch. It fixes the "close" icon issue.

Please take a look at this comment: https://git.drupalcode.org/project/rigel/-/merge_requests/68#note_370892

Aside from that, it is looks good to me.

@sachin.addweb, Thank you for the updated patch. I made one code comment. Also, please see attached screenshot. When you added the new "X" as the close icon, there smaller one is still showing up.

Changing to needs review to trigger testing.

@sachin.addweb, Thank you for taking over this patch. There are stylelint issues reported with main.css during Unit testing. Can you please run Step 6 from the README.md, npx prettier --config .prettierrc.json --write "assets/css/main.css". That will fix all the stylelint issues.

@yas, thank you for the update on the patch! It looks good to me now.

@yas, thank you for the patch. I just have these minor comments. Aside from that, it looks good.

@yas, thank you for the patch. This looks good to me.

@yas this looks good to me. Thank you for the patch.

@yas this looks good to me. Thank you for the patch.

@yas, thank you for the patch. This looks good to me.

@yas, thank you for the patch. It looks good to me.

@yas, please review the updated patch.

@yas please review the patch.

baldwinlouie → created an issue.

@yas, thank you for the updated patch. It looks good to me.

@yas, thank you for the patch. I have one comment.

@yas Please review this patch. This takes care of a few things

1. Fixes the white background because in Drupal 10.4+, the css class is accordion and not card anymore
2. Changes all double quotes to single quote because of Drupal coding standards reported by prettier
3. Moves css attributes around reported by prettier. For example, all transition attributes need to be moved lower in a css class
4. Setup Grunt so it automatically runs a local prettier to reformat all code based on the .prettierrc.json

@yas, thank you for the patch. It looks good to me.

@yas, thank you for the patch. It looks good to me.