Skip to content

Refactor SearchPage(s)#815

Merged
Gnito merged 7 commits intomainfrom
refactor-SearchPage
Apr 14, 2026
Merged

Refactor SearchPage(s)#815
Gnito merged 7 commits intomainfrom
refactor-SearchPage

Conversation

@Gnito
Copy link
Copy Markdown
Contributor

@Gnito Gnito commented Mar 27, 2026

Explore options to reduce duplicated code between search page variants.

  • Add custom component to handle access control redirections (SearchPageAccessWrapper)
  • Move shared functions and derived data to SearchPage.shared.js file
    • getDerivedRenderData
    • onApplyFilters
    • onSortBy
    • onResetAll
    • createFilterValueChangeHandler
  • Remove deprecated Redux functions: connect, mapStateToProps and mapDispatchToProps

@Gnito Gnito changed the title SearchPage: refactoring Refactor SearchPage Mar 27, 2026
@Gnito Gnito marked this pull request as draft March 27, 2026 06:42
@Gnito Gnito force-pushed the refactor-AuthenticationPage branch from 4c85c31 to 61a4b6b Compare April 7, 2026 15:10
@Gnito Gnito force-pushed the refactor-SearchPage branch from a5eb189 to afee173 Compare April 7, 2026 15:12
@Gnito Gnito force-pushed the refactor-AuthenticationPage branch from 61a4b6b to 8dd491a Compare April 8, 2026 08:40
@Gnito Gnito force-pushed the refactor-SearchPage branch from afee173 to a7b26ff Compare April 8, 2026 08:42
@Gnito Gnito marked this pull request as ready for review April 8, 2026 09:48
@Gnito Gnito changed the title Refactor SearchPage Refactor SearchPage(s) Apr 9, 2026
@Gnito Gnito force-pushed the refactor-AuthenticationPage branch from 8dd491a to b95333e Compare April 14, 2026 10:03
Base automatically changed from refactor-AuthenticationPage to main April 14, 2026 10:09
@Gnito Gnito force-pushed the refactor-SearchPage branch from a7b26ff to 3edb715 Compare April 14, 2026 10:12
@Gnito Gnito merged commit 048b2cd into main Apr 14, 2026
3 checks passed
@Gnito Gnito deleted the refactor-SearchPage branch April 14, 2026 10:16
@Gnito Gnito mentioned this pull request Apr 14, 2026
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.

1 participant