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

chore(edit-content): include language field for search in relationships field #31253

Merged

Conversation

nicobytes
Copy link
Contributor

@nicobytes nicobytes commented Jan 28, 2025

Parent Issue

#31127

Proposed Changes

This pull request introduces a new LanguageFieldComponent for selecting languages in a dropdown menu. It includes the component's implementation, its associated store for managing state, and comprehensive unit tests for both the component and the store.

New Component and Store Implementation:

  • Component Implementation:

    • language-field.component.html: Added HTML template for the language dropdown, including labels, dropdown options, and error handling.
    • language-field.component.ts: Defined the LanguageFieldComponent with Angular's ControlValueAccessor and OnInit interfaces, including methods for handling language selection and form control integration.
  • Store Implementation:

    • language-field.store.ts: Created LanguageFieldStore using NgRx signals for state management, including methods for loading languages, selecting a language, and handling errors.

Unit Tests:

  • Component Tests:

  • Store Tests:

    • language-field.store.spec.ts: Added unit tests for the LanguageFieldStore, covering initial state, state management, language selection, error handling, and reset functionality.

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (add notes if applicable)

This PR fixes: #31127

@nicobytes nicobytes marked this pull request as ready for review January 28, 2025 20:24
@nicobytes nicobytes requested a review from Copilot January 28, 2025 20:24
@nicobytes nicobytes changed the title chore(edit-content): include language-field chore(edit-content): include language field for search in relationships field Jan 28, 2025
@nicobytes nicobytes requested review from oidacra and hmoreras January 28, 2025 20:39
@nicobytes nicobytes enabled auto-merge January 29, 2025 15:28
@nicobytes nicobytes added this pull request to the merge queue Jan 29, 2025
Merged via the queue into main with commit 69f3532 Jan 29, 2025
36 checks passed
@nicobytes nicobytes deleted the 31127-relationship-field-enable-static-fields-for-search branch January 29, 2025 20:00
@nicobytes nicobytes linked an issue Jan 31, 2025 that may be closed by this pull request
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.

[EPIC] Relationship Field: Enable Static Fields for Search Relationship Field Search: Language Field
3 participants