- Issue created by @mikelutz
- Assigned to annmarysruthy
- Merge request !8924Fix issue #3462868: replace http with https in reference links โ (Open) created by annmarysruthy
- Issue was unassigned.
annmarysruthy โ changed the visibility of the branch 3462868-replacehttp-1 to hidden.
- Status changed to Needs review
about 1 month ago 1:23pm 25 July 2024 - Status changed to Needs work
about 1 month ago 4:57pm 31 July 2024 - ๐บ๐ธUnited States smustgrave
Seems to be some missed one
views.theme.inc
breakpoint.module
breakpoint.overview.html.twigmaybe a few more.
- First commit to issue fork.
- Status changed to Needs review
about 1 month ago 11:40am 1 August 2024 - Status changed to Needs work
about 1 month ago 3:12pm 1 August 2024 - ๐ฎ๐ณIndia ankitv18
I found this reference http://www.w3.org in total 80 files and MR consists of changes in 52 files
Seems more files need to be taken care of. - Assigned to NexusNovaz
- ๐ฌ๐งUnited Kingdom NexusNovaz
Ill take a quick look and see if i can get the rest
- Issue was unassigned.
- Status changed to Needs review
about 1 month ago 2:32pm 3 August 2024 - ๐ฌ๐งUnited Kingdom NexusNovaz
Found two more files which matched the problem. There is one last file which is contained in ckeditor5-dll.js. As this wasn't a normal file, i've left that one to be discussed. There are another 100 matches however, those are xmlns references.
- ๐ฎ๐ณIndia ankitv18
Changes looks good but not sure whether we need to consider .twig .php files having http://www.w3.org (Within svg xmlns)
Attaching screenshot for your reference.
- ๐บ๐ธUnited States smustgrave
Think svgs should be a novice follow up
- Status changed to Needs work
about 1 month ago 5:02pm 3 August 2024 - ๐บ๐ธUnited States smustgrave
Also @ankitv18 please leave this for the newer users who were already working on it thanks
- ๐ฎ๐ณIndia ankitv18
@smustgrave, I just reviewed and synced the forked branch
- Assigned to Yujiman85
- Issue was unassigned.
- Status changed to Needs review
22 days ago 2:19pm 16 August 2024 - ๐บ๐ธUnited States Yujiman85
I wasn't sure if the .js files needed to be touched since those seemed compiled and minified. Please correct me if I'm wrong. This needs a review.
- Status changed to Needs work
19 days ago 12:22pm 20 August 2024 - ๐บ๐ธUnited States smustgrave
Welcome @Yujiman85 and nice work!
Appears to need a rebase now though. Will keep an eye out for your changes.
- ๐ณ๐ฟNew Zealand quietone New Zealand
This is still a very large change. I am guessing a script is used to do this? Can it be shared?
- Status changed to Needs review
15 days ago 8:28pm 23 August 2024 - ๐บ๐ธUnited States Yujiman85
@smustgrave Alright, I believe I did this right. I rebased the branch and it is up to date now with the commits from this branch. Needs a review.
- Status changed to Needs work
7 days ago 4:21am 1 September 2024 - ๐ฎ๐ณIndia ankitv18
Pipelines arenโt passing ~~ hence moving back into NW