Thanks for the interest in helping maintaining and getting this project to Drupal 10+.
Added the following maintainers:
- aaron.ferris
- apaderno
- bburg
I believe that Norah would be a valuable addition to the Project Update Working Group and would love to see her use her talents to further Drupal. Norah is a skilled technical leader with a background in both development and project management. In addition, she has contributed to the Drupal community through mentorship and volunteering at camps. I am confident that providing Norah with this opportunity to contribute more would allow her to continue on her path of growth, while improving the Drupal Community.
Good to hear, @Kristen Pol! Thanks for offering to help!
Kristen, you have been added.Thanks for helping!
volkswagenchick → credited shrop → .
Thank to the team here! Great to get this going and talk next steps.
Guardr Core 2.1.0 → released.
Thanks Joe and Gareth!
I like this and noting we did remove Hacked in this discussion. I think once we have the D10 version released, we can look at the other items around splitting things up more and some other upcoming changes.
I am also thinking we should do Drupal core reqs like this - ^9.3 || ^10 vs. ^9 || ^10 could protect us from dependencies requiring 9.3. We have some now.
Committed and released with 2.1.0-beta1 → . I set this as a new major release since it supports a new version of Drupal. The 2.0.0-beta1 release is no longer supported.
Thanks for helping test this out!
lol I think I know why. This is for the D7 version. I can take another look at this!
Thanks for the work on this everyone!
I can't get the latest patch to apply. If someone can figure out why, that would be great.
➜ curl
https://www.drupal.org/files/issues/2022-02-08/3047456-17.patch →
| git apply
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
100 2716 100 2716 0 0 4582 0 --:--:-- --:--:-- --:--:-- 4619
error: patch failed: README.txt:1
error: README.txt: patch does not apply
Could also try a merge requst.
@Grienauer, can you provide more details on this solution you found?
What does the Microsoft app URL mean exactly? I am seeing similar issue with users who simply changed MFA providers.
Thanks for the updates @apmsooner. We will test this as a next step.
For the "Only trigger builds for published content" not being set, what would be a process to ensure live content isn't published or is it covered via Fastbuilds publish/preview payloads?
Definitely a good plan. Not putting as much effort into D7 stuff now. So closing.
Patched merged and we can do some prelimiary D10 testing now against 2.x-dev.
Merged https://git.drupalcode.org/project/guardr_core/-/merge_requests/8
Thanks all!
Merged in https://git.drupalcode.org/project/guardr_core/-/merge_requests/9
Thanks all!
Appreciate all the work in this issue. I think we need to work on some changes and this covers many of them. What do you think about working on this for a 3.x version of Guardr Core? Seems like that would be a good break. Right now, I would like to get Guardr 2.x to D10 compatibility and then we could do this as a D10 compatible version at 3.x.
Looks good. We can eval dropping 1.1 version at some point. For now, merged and moving forward! https://git.drupalcode.org/project/guardr_core/-/merge_requests/10
Agree with ya'll that Guardr Core is really exactly that in Drupal 8, 9, and 10; a module to install Guardr with any Drupal install. Closing this one. Thanks!
Resolved in 📌 Please add D10 version Fixed
Updates
- MR has been merged into the 8.x-1.2 branch
- Created release 8.x-1.3 → to include the changes
- That's the last release on this branch since 2.0.0 will be the new supported release
- 2.x-dev release →
- 2.x branch supporting Drupal 9.4 and up, including Drupal 10
- 2.0.0 supported release →
Thanks for the help everyone! Let me know if you have questions or see any issues.
📌 Please add D10 version Fixed
volkswagenchick → credited shrop → .