Jump to content
  • Donations Changes


    Roy
    • Start Date: 05/04/2018
      Completion Date: 05/29/2018
      Expected Completion Date: 05/29/2018
     Share

    Deadline is approaching for GFL Donations app to be updated to work with PayPal's new API. We should get this done ASAP.

     

    In addition, we should also restructure the donation packages.

     

    This is a project moved from our old Technical Tasks database to the Projects database. Some sensitive information may have been cut out when moving to the public.

    Edited by Roy


    Leader: Roy Members - Roy
    Progress - 100%
     Share


    User Feedback

    Recommended Comments

    5-4-18

    Only thing we will need to change on our end will be the GET forms to POST.

     

    Thanks.

     

    5-4-18

    I would also like to note these changes shouldn't be difficult to make. Currently, the GET function used is the following:

     

    	/**
    	 * HTTP GET
    	 *
    	 * @return	\IPS\Http\Response
    	 * @throws	\IPS\Http\Request\CurlException
    	 */
    	public function get()
    	{
    		/* Specify that this is a GET request */
    		curl_setopt( $this->curl, CURLOPT_HTTPGET, TRUE );
    		$this->dataForLog = NULL;
    					
    		return $this->_executeAndFollowRedirects( 'GET', NULL );
    	}

     

    We will have to switch it to this function:

     

    	/**
    	 * HTTP POST
    	 *
    	 * @param	mixed	$data	Data to post (can be array or string)
    	 * @return	\IPS\Http\Response
    	 * @throws	\IPS\Http\Request\CurlException
    	 */
    	public function post( $data=NULL )
    	{		
    		/* Specify that this is a POST request */
    		curl_setopt( $this->curl, CURLOPT_POST, TRUE );
    		
    		/* Set the data */
    		curl_setopt( $this->curl, CURLOPT_POSTFIELDS, $this->_dataToSend( $data ) );
    		
    		/* Execute */
    		return $this->_executeAndFollowRedirects( 'POST', $data );
    	}

     

    We will basically just have to store the data differently through POST. In this case, through the post() function itself as a parameter.

     

    Thanks.

     

    5-24-18

    I will be working on this soon.

     

    5-26-18

    I have implemented these changes on our beta website. I would like to find a way to test the changes out before deploying to the live website.

     

    Thank you.

     

    5-29-18

    Will be most likely testing and implementing this today.


    Thank you.

     

    5-29-18

    These changes have been implemented and appear to be working according to my test donation on my alt Steam account.

     

    I will keep an eye on the next few donations to ensure they're getting their perks correctly.

     

    Thanks!

     

    5-29-18

    Closing task.

    Link to comment
    Share on other sites


    Guest
    This is now closed for further comments

×
×
  • Create New...