Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Initial page add for url searching. #420

Merged
merged 4 commits into from
Nov 22, 2019

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Nov 9, 2019

Preliminary page on url searching - I could do with some help on how to make this work - it might get quite
busy as I work on other screens / fix the ones I couldn't add as yet. Formatting is kinda missing

See https://github.com/civicrm/civicrm-dev-docs/issues/624

Also in my testing I just opened
civicrm/civicrm-core#15787

Other than that the following test OK on contribution search as urls - ie. high, low & relative variants

'contribution_recur_start_date',
  'contribution_recur_next_sched_contribution_date',
  'contribution_recur_cancel_date',
  'contribution_recur_end_date',
  'contribution_recur_create_date',
  'contribution_recur_modified_date',
  'contribution_recur_failure_retry_date',

@seamuslee001 @MikeyMJCO @monishdeb @lcdservices

Preliminary page on url searching - I could do with some help on how to make this work - it might get quite
busy as I work on other screens / fix the ones I couldn't add as yet. Formatting is kinda missing
@seamuslee001
Copy link
Collaborator

seamuslee001 commented Nov 15, 2019

Comment on wrong PR

@homotechsual
Copy link
Contributor

Added some formatting changes - are we envisioning this expanded for Contribution searches, Contact searches etc?

@seamuslee001
Copy link
Collaborator

@MikeyMJCO yes, at the moment it has coverage for the Find Contributions and FInd Participants screens and also advanced search for those two sections in the advanced search. It would be anticipated that more fields are added as we convert them over to being built off the metadata. would be my expectation @eileenmcnaughton ?

@homotechsual
Copy link
Contributor

Merging this as is for now - can be expanded in future PRs.

@homotechsual homotechsual merged commit 38c3be8 into civicrm:master Nov 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants