Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge branch '7' into 8 #878

Merged
merged 53 commits into from
Aug 20, 2024
Merged

Merge branch '7' into 8 #878

merged 53 commits into from
Aug 20, 2024

Conversation

GuySartorelli
Copy link
Contributor

@GuySartorelli GuySartorelli commented Aug 20, 2024

This is a straight merge from the 7 branch to the 8 branch. There's no merge conflicts so I was able to create the PR directly without needing a fork in the middle.

GuySartorelli and others added 30 commits February 17, 2023 14:08
skip empty 'where' clauses
Also copy translations on duplicate
Implement onAfterDuplicate
feat: add absolute base url for locales
copy Bugfix for GridFieldAddExistingSearchButton from master to version 6.0
fixes #823 - The magic constant __TRAIT__ was inserting namespaces into
localization keys, containing slashes that could not be parsed by the
TextCollector task. These have been replaced w/ the Trait name as a
string
FIX allows i18nTextCollector to concatenate keys
fix: don't invoke db for ready check outside of dev urls
# Conflicts:
#	src/Extension/FluentExtension.php
fix: check for request prior to access
New task is based on the original task built for SiteTree but is now extendable for any DataObject enabled with Fluent. To prevent duplicate code, the original task now extends the new task.
…taobjects

New dev/task to initialise DataObjects for existing datasets
emteknetnz and others added 23 commits May 24, 2024 12:40
FIX Issues when running with other modules installed
sass-lint went EOL ages ago
@tractorcow
Copy link
Collaborator

exciting. Approved from my pov. are you happy to merge when you're ready?

@GuySartorelli
Copy link
Contributor Author

Yup, merging now. Thanks!

@GuySartorelli GuySartorelli merged commit de6c50b into 8 Aug 20, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants