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

Contribution url params - add contribution_page_id as a supported url parameter #15785

Merged

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Nov 8, 2019

Overview

Makes it possible to put contribution_page_id=1 as a search param, deprecates the old variant

Before

contribution_page_id=1 does not work
Screen Shot 2019-11-11 at 6 22 37 PM

After

contribution_page_id=1 does work

Screen Shot 2019-11-11 at 6 23 37 PM

Technical Details

Part of a generalised standardisation

Comments

I'm not planning on doing more conversions on this page at the moment - I did this primarily in the context of auditing for civicrm/civicrm-user-guide#420

@seamuslee001 this highlights a formatting issue ... oh - it's not select-2 by default - I don't like that!

@civibot
Copy link

civibot bot commented Nov 8, 2019

(Standard links)

@civibot civibot bot added the master label Nov 8, 2019
@eileenmcnaughton eileenmcnaughton force-pushed the contribution_url_params branch 4 times, most recently from 2a2734b to 07cd47d Compare November 11, 2019 05:18
@eileenmcnaughton eileenmcnaughton changed the title [WIP] Contribution url params - pending other merges Contribution url params - add contribution_page_id as a supported url parameter Nov 11, 2019
@eileenmcnaughton
Copy link
Contributor Author

@seamuslee001 @colemanw should we be defining 'Select2' in the xml for contribution page ID? I guess it makes sense but it feels like one everything will be & is_select2 would make more sense

@seamuslee001 seamuslee001 added the merge ready PR will be merged after a few days if there are no objections label Nov 15, 2019
@seamuslee001
Copy link
Contributor

(CiviCRM Review Template WORD-1.2)

  • General standards
  • Developer standards

Added Merge ready flag to give Coleman a chance to comment re select2

@seamuslee001
Copy link
Contributor

I'm going to merge this now

@seamuslee001 seamuslee001 merged commit 76f7a8b into civicrm:master Nov 21, 2019
@seamuslee001 seamuslee001 deleted the contribution_url_params branch November 21, 2019 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants