Hi elber,
I applied and reviewed your MR and confirmed that README.md is now present and following the README.md template. I have made the follwing steps while testing:
- I have first clone the module in my locally.
- Then I download the patch from the merge request.
- Applied the patch on the module and check the file structure.
- The readme.md file was present in the folder structure.
- Opened the Readme.md filea and check whether all the spacing is properly done.
- Everything ws fine and all the informatin was present in the file.
Everything looks very well. No issue noticed from my end.
Arshu1864 โ made their first commit to this issueโs fork.
I have reviewed the patch and it is working finre the module is succesfully installed on drupal 10 working perfectly fine. I have done the following steps to tet the patch:
- Install drupal 10 fresh.
- Clone the module.
- Use wgwt to download the patch.
- Use git apply-index and patch file to apply the patch.
- Install the patch and check whether it is working fine or not and whether this patch is creating some issue or not.
I have created a merge request and changed the code please review it.
Arshu1864 โ made their first commit to this issueโs fork.
I have fixed some of the coding standard issue on the module and created merge request for it but some dependecies need to be resolved from the core.
The issue has been resolved the theme is now compatible with drupal 10 please test it.
I have tested the theme and the menus are working fine on mobile and desktop devices. I have done the following steps to test this issue:
- Install the latest version of drupal 9.
- Install the theme using composer.
- Then I observe the issues carefully.
- After observing the issue I pulled the merge request from the git.
- I see and tested the changes done after the merge request was pulled.
- The menu was working fine after I pulled the code from merge request.
Now when the issue is fixed it can pushed to RTBC
https://git.drupalcode.org/project/cog/-/merge_requests/2
This is the merge request I have created please check
Sorry the while creating patch file the # got added I had renamed the file and added along with this comment
I have cloned from the 8.x-1.x branch and my npm version is 9.6.4 and node version is v16.19.1
Arshu1864 โ created an issue.
I have fixed the issue and tested it now it is working fine. There is on npm audit there is showing found 0 vulnerabilities. I am attaching patch alongside with it please check.
I'm fixing the issue and will hopefully be resolved by end of this week
I take full responsibility of solving this issue
I have applied the patch now the theme is compatible with druapl 10. Please test it
Arshu1864 โ created an issue.
I want to be the maintaniner of this theme and I will take full responsibility of mainataining this theme on regular basis
I have migrated the theme to drupal 10 and its subtheme also please test it.
Arshu1864 โ created an issue.
I have Fixed the issue change the core version requirement from ^9.5 to ^9.
Summiting a patch please review.