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

3.0.0-alpha3 #290

Merged
merged 19 commits into from
Jul 24, 2023
Merged

3.0.0-alpha3 #290

merged 19 commits into from
Jul 24, 2023

Conversation

ekes
Copy link
Member

@ekes ekes commented Jun 22, 2023

Update module dependencies for D10.

stephen-cox and others added 4 commits June 22, 2023 14:42
Doing dev on a channel that doesn't use content facet, but taxonomy
facets instead, and keep getting reminded this field isn't actually
compulsory. So figured rather than unticking it every time I reinstall I
should make a PR :)
…tional

Make facets on a channel optional, as is documented from #281.
@finnlewis
Copy link
Member

Merge Monday discussion: we should probably fix the deprecated code issues.
(But note that the 3.x branch of localgov is currently pointing at dev branch for localgov_directories.)

@finnlewis finnlewis self-assigned this Jul 3, 2023
@stephen-cox
Copy link
Member

I've started fixing things on #294

@stephen-cox
Copy link
Member

Still one failing test.

Fails because the 'settings[handler_settings][target_bundles][localgov_directory]' field doesn't exist in the page with:

https://github.com/localgovdrupal/localgov_directories/blob/3.x/tests/src/Functional/ChannelFacetsWidgetAdminTest.php#L122-L131

It exists as a hidden field on my local, but is missing in the test. Been bashing my head against it, but no luck so far

@finnlewis
Copy link
Member

Note: we already have an alpha2, this might want to be alpha3

stephen-cox and others added 2 commits July 13, 2023 14:41
(a) the new 're-use existing field' form doesn't suffer the JS switch of
  form that create field does. So the entity types don't appear on that
  form when they still do for create.
(b) the new form settings field has a toggle for enabling a default
  value.
@ekes
Copy link
Member Author

ekes commented Jul 13, 2023

I've just pushed the fix for the test fails into e9dbc2e can be cherry picked to here if we want to get this one out first.

@finnlewis finnlewis changed the title 3.0.0-alpha2 3.0.0-alpha3 Jul 24, 2023
@finnlewis finnlewis merged commit 3a8cba5 into release-3.x Jul 24, 2023
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants