Updating from SendGrid V2 API to V3

Issue Summary
I'm trying to update my website, hosted on a 3rd party server, from V2 SendGrid API to V3. I downloaded the zip package from GitHub and unpacked into my testing area on my local Windows PC. I wrote a simple test application based upon code as in https://github.com/sendgrid/sendgrid-php#usage "Hello Email" example, only modifying the to and from data and adding the API key I downloaded earlier. I don't use composer as I don't know if it is installed on the 3rd party and I don't have access to command line. The example implies that you don't need composer.
I assume that I have to delete the entire contents of the V2 sendgrid-php folder and replace it with the files downloaded from GitHub - there does not appear to be any documentation on this. There are no instructions otherwise here https://sendgrid.com/docs/Classroom/Send/v3_Mail_Send/how_to_migrate_from_v2_to_v3_mail_send.html
If I run the app I get the error as in step 1 below
If I look in the sendgrid-php there is no vendor folder
I then copied the vendor folder from my V2 download into the sendgrid-php folder
Steps to Reproduce
Step 1 - Install library from GitHub exactly as unpacked
Install the library
Add the API Key to the example
Change e-mail addresses to ones that should work
Run app
Error
Warning: require(........sendgrid-php/vendor/autoload.php): failed to open stream: No such file or directory in .........sendgrid-php\sendgrid-php.php on line 2
Fatal error: require(): Failed opening required '...........sendgrid-php/vendor/autoload.php' (include_path='.;C:\php\pear') in .........................................sendgrid-php\sendgrid-php.php on line 2
Step 2 - Install library from GitHub with additional vendor folder
Install the library
Add the vendor folder from V2
Add the API Key to the example
Change e-mail addresses to ones that should work
Run app
Error
Fatal error: Uncaught Error: Class 'SendGrid\Email' not found in ............php:93 Stack trace: #0 {main} thrown in ............php on line 93
In the V2 folder there is a SendGrid/Mail.php in the lib folder. In the V3 folder there is a SendGrid.php file but no SendGrid folder with an Email.php file
I don't think I want to do any more fiddling with the code. I'm surprised that doing something as basic as this causes these sort of errors
Having looked at the documentation I hope that the library provides a simpler method of handling attachments than described. Like on of your other correspondents, I've too much to do to worry about digging in the depths of the API. I want to just be able to use a straightforward interface - which the V2 API provided
Technical details:

sendgrid-php Version: master (latest commit: 6.0.0 as in changelog.md)
PHP Version: 7.1.3

Hi @ianh2,
Instead of including vendor/autoload.php, could you please instead try including this file in your test project?
Thanks!
Elmer

Hi Elmer
I modified the sendgrid-php.php file from

To

I ran my test file (based on your example) and got the following in my server's error log
[Tue Jul 04 09:13:18 2017] [error] [client <IP_ADDRESS>:34358] AH01215: PHP Warning:  require_once(/home/sites/.../public_html/sendgrid-php/SendGrid.php): failed to open stream: No such file or directory in /home/sites/.../public_html/sendgrid-php/sendgrid-php.php on line 5: /home/sites/..../public_html/testarea/...TestMail.php
[Tue Jul 04 09:13:18 2017] [error] [client <IP_ADDRESS>:34358] AH01215: PHP Fatal error:  require_once(): Failed opening required '/home/sites/.../public_html/sendgrid-php/SendGrid.php' (include_path='.:/usr/share/pear55:/usr/share/php') in /home/sites/.../public_html/sendgrid-php/sendgrid-php.php on line 5: /home/sites/.../public_html/testarea/...TestMail.php
There's no SendGrid.php file in the sendgrid-php folder or no helpers folder so I altered sendgrid-php.php to

This took me a bit further and I got the following error - this was displayed on the web page
Fatal error: Class 'SendGrid\Client' not found in /home/sites/.../public_html/sendgrid-php/lib/SendGrid.php on line 53
There is no 'Client.*' file in the GitHub download
Note that the V2  sendgrid-php.php is as follows and that works with no problems

Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 04 July 2017 00:46
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hi @ianh2 https://github.com/ianh2 ,
Instead of including vendor/autoload.php, could you please instead try including this file https://github.com/sendgrid/sendgrid-php/blob/master/lib/loader.php  in your test project?
Thanks!
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-312750601 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6nSix3qPvwbTl0HQH8cj-qpvM5lLks5sKX0ZgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6vFtrh97syUxTsWl-cULe0IQwv4kks5sKX0ZgaJpZM4OL-VK.gif

Hi @ianh2,
It sounds like you may be trying to use v2 with v3. This SDK will not work with v2. You need to choose one or the other. Please remove all files relating to v2, download this SDK, then try my suggestion.
I hope that helps!
With Best Regards,
Elmer

So what you're saying is that there is no transition available in the change from V2 to V3?  This means that I cannot convert my pages one at a time. I have to install V3 and hope that everything works first time?
Cannot I create a sendgrid3 folder and change the reference for each page that needs changing? This would allow a graceful transition.
A lack of any sort of backward compatibility or transition does seem to indicate user unfriendly software
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 04 July 2017 13:53
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hi @ianh2 https://github.com/ianh2 ,
It sounds like you may be trying to use v2 with v3. This SDK will not work with v2. You need to choose one or the other. Please remove all files relating to v2, download this SDK, then try my suggestion.
I hope that helps!
With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-312870447 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6s39LW4uK_c2uHVZ8UPoVtjRFND_ks5sKjWdgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6q4RRsic4QQCM0ZR6X9MEYK4vFFDks5sKjWdgaJpZM4OL-VK.gif

I did the following

Removed the sendgrid-php folder and its contents from the live server (hoping that no-one would want to use a mail function)

Uploaded the contents of the GitHub download to a new sendgrid-php folder

Amended the sendgrid-php.php file as recommended in your e-mail

Modified the  sendgrid-php.php file to prefix both require targets with "/lib"

Ran the program

Same error as before
Fatal error: Class 'SendGrid\Client' not found in /home/sites/...../public_html/sendgrid-php/lib/SendGrid.php on line 53
As I said before there is no "Client" folder or file in the GitHub V3 download. There is a "clients" in examples
Ian
PS I then reinstated V2 and deleted V3 and all is back to normal
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 04 July 2017 13:53
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hi @ianh2 https://github.com/ianh2 ,
It sounds like you may be trying to use v2 with v3. This SDK will not work with v2. You need to choose one or the other. Please remove all files relating to v2, download this SDK, then try my suggestion.
I hope that helps!
With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-312870447 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6s39LW4uK_c2uHVZ8UPoVtjRFND_ks5sKjWdgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6q4RRsic4QQCM0ZR6X9MEYK4vFFDks5sKjWdgaJpZM4OL-VK.gif

@ianh2,
Thanks for the follow up. I see the issue now. You are missing this dependency: https://github.com/sendgrid/php-http-client.
I will need to update the non-composer install instructions, this is now on our backlog.
Thanks!
With Best Regards,
Elmer

OK - I'll have a go at retrying later on
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 05 July 2017 16:55
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
@ianh2 https://github.com/ianh2 ,
Thanks for the follow up. I see the issue now. You are missing this dependency: https://github.com/sendgrid/php-http-client.
I will need to update the non-composer install instructions, this is now on our backlog.
Thanks!
With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-313145869 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6v_1XpyXpMOF_BnJCHUmP38pg9zqks5sK7HDgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6kXRk0icQ0PXKJ9b5ka6Vw1B9lkgks5sK7HDgaJpZM4OL-VK.gif

Elmer
I'm not sure you do!
I managed to get back to the error by doing

Clear the contents of the sendgrid-php folder

Copy in the original Git Hub download

Copy on top, the link you refer to below

If you don't do this then at least the following files are missing from your new download
sendgrid-php.php
lib/SendGrid.php

This gives an error, so I modified the sendgrid-php.php file as stated yesterday

 ,

Thanks for the follow up. I see the issue now. You are missing this dependency: https://github.com/sendgrid/php-http-client.

I will need to update the non-composer install instructions, this is now on our backlog.

Thanks!

With Best Regards,

Elmer

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub  , or mute the thread  .

'SendGrid\Client' is contained in this dependency: https://github.com/sendgrid/php-http-client

Yes but not SendGrid.php. I installed SendGrid.php from the original download
The Client was installed in the test system. Is it not expecting to find a file "Clint" in a "SendGrid" folder?
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 05 July 2017 19:53
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
'SendGrid\Client' is contained in this dependency: https://github.com/sendgrid/php-http-client
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-313193643 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6taryYLK2E86pKGqEeE_0iiuEYbXks5sK9t3gaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6ln_rZe6i2tTwPJQUQJD1bXMsa85ks5sK9t3gaJpZM4OL-VK.gif

Hi @ianh2,
I want to try and reproduce the error locally. Can you please tell me the directory structure of your code. Even better if you can share the relevant codebase with me at<EMAIL_ADDRESS>At least enough to try and reproduce the error.
With Best Regards,
Elmer

Hi Elmer
I've sent you all relevant files in an email to the address you provided. It does contain a zip file which might get spam trapped
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 06 July 2017 00:46
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hi @ianh2 https://github.com/ianh2 ,
I want to try and reproduce the error locally. Can you please tell me the directory structure of your code. Even better if you can share the relevant codebase with me at<EMAIL_ADDRESS>At least enough to try and reproduce the error.
With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-313256573 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6hwRdbXNsEtQzOch9PYacAWeZYuIks5sLCA0gaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6t-CwDaVtUhlgN3xuoHkesOVHCPlks5sLCA0gaJpZM4OL-VK.gif

Hi @ianh2,
I have just responded via email, please try out my suggestion and let me know how it goes. Thanks!

Hi Elmer
Sorry but I can't do anything today as I've got to go to see a client
Will look tomorrow
Ian
-------- Original message --------From: Elmer Thomas<EMAIL_ADDRESS>Date: 06/07/2017  23:23  (GMT+00:00) To: sendgrid/sendgrid-php<EMAIL_ADDRESS>Cc: ianh2<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
  (#412)
Hi @ianh2,
I have just responded via email, please try out my suggestion and let me know how it goes. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/sendgrid/sendgrid-php","title":"sendgrid/sendgrid-php","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/sendgrid/sendgrid-php"}},"updates":{"snippets":[{"icon":"PERSON","message":"@thinkingserious in #412: Hi @ianh2,\r\n\r\nI have just responded via email, please try out my suggestion and let me know how it goes. Thanks!"}],"action":{"name":"View Issue","url":"https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-313535603"}}}

Great, thanks!

I was wondering if there was any update on this. I myself am having this same issue, where "Class 'SendGrid\Client' not found". All I have is the base download of the sendgrid-php codebase and I have changed the require statement to: require("sendgrid-php/lib/loader.php");.

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
I was wondering if there was any update on this. I myself am having this same issue, where "Class 'SendGrid\Client' not found". All I have is the base download of the sendgrid-php codebase and I have changed the require statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314361038 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCgawks5sMyZlgaJpZM4OL-VK.gif

I ran into the same issue. I think that issue with the verification is due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I was wondering if there was any update on this. I myself am having this
same issue, where "Class 'SendGrid\Client' not found". All I have is the
base download of the sendgrid-php codebase and I have changed the require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314370150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK
.

Thanks!
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
I ran into the same issue. I think that issue with the verification is due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I was wondering if there was any update on this. I myself am having this
same issue, where "Class 'SendGrid\Client' not found". All I have is the
base download of the sendgrid-php codebase and I have changed the require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314370150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314371168 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6kuMy_nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date: Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain; charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6] => X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] => Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] => Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers: Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] => Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason: https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14] => )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
I ran into the same issue. I think that issue with the verification is due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I was wondering if there was any update on this. I myself am having this
same issue, where "Class 'SendGrid\Client' not found". All I have is the
base download of the sendgrid-php codebase and I have changed the require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314370150,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314371168 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6kuMy_nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif

Awesome. Glad I could help out a little bit.
On Tue, Jul 11, 2017 at 1:39 AM, ianh2<EMAIL_ADDRESS>wrote:

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date:
Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain;
charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6] =>
X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] =>
Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] =>
Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers:
Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] =>
Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason:
https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14] => )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I ran into the same issue. I think that issue with the verification is due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I was wondering if there was any update on this. I myself am having this
same issue, where "Class 'SendGrid\Client' not found". All I have is the
base download of the sendgrid-php codebase and I have changed the require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314370150>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314371168>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/
AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6kuMy_
nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314374781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_gyfN1GtaaWNSks5sMzSpgaJpZM4OL-VK
.

Interesting. I get the error message but the mail still gets sent. I wasn't expecting that
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:41
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Awesome. Glad I could help out a little bit.
On Tue, Jul 11, 2017 at 1:39 AM, ianh2<EMAIL_ADDRESS>wrote:

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date:
Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain;
charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6] =>
X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] =>
Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] =>
Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers:
Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] =>
Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason:
https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14] => )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I ran into the same issue. I think that issue with the verification is due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I was wondering if there was any update on this. I myself am having this
same issue, where "Class 'SendGrid\Client' not found". All I have is the
base download of the sendgrid-php codebase and I have changed the require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412# <https://github.com/sendgrid/sendgrid-php/issues/412#>
issuecomment-314370150>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314371168>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6kuMy_
nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314374781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_gyfN1GtaaWNSks5sMzSpgaJpZM4OL-VK
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314375239 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6kXIpEWxhPBBX9gv-FiXNp74LqIZks5sMzUQgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6tNQEbMwweSApjZV_uVih3r1kuCtks5sMzUQgaJpZM4OL-VK.gif

I dont believe it is an error message. That is just a response that the API
sends back. Yours looks like it was a success and accepted. if you remove
or comment out the following lines, you should not get any response other
than your email being sent:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();
On Tue, Jul 11, 2017 at 1:57 AM, ianh2<EMAIL_ADDRESS>wrote:

Interesting. I get the error message but the mail still gets sent. I
wasn't expecting that
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:41
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
Awesome. Glad I could help out a little bit.
On Tue, Jul 11, 2017 at 1:39 AM, ianh2<EMAIL_ADDRESS>wrote:

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date:
Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain;
charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6]
=>
X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] =>
Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] =>
Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers:
Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] =>
Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason:
https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14]
=> )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I ran into the same issue. I think that issue with the verification is
due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to
V3
(#412)
I was wondering if there was any update on this. I myself am having
this
same issue, where "Class 'SendGrid\Client' not found". All I have is
the
base download of the sendgrid-php codebase and I have changed the
require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe-
auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412# <
https://github.com/sendgrid/sendgrid-php/issues/412# >
issuecomment-314370150>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ <
https://github.com/notifications/unsubscribe-auth/>
ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314371168>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6kuMy_
nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314374781>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_
gyfN1GtaaWNSks5sMzSpgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314375239>
, or mute the thread <https://github.com/notifications/unsubscribe-
auth/AceR6kXIpEWxhPBBX9gv-FiXNp74LqIZks5sMzUQgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6tNQEbMwweSApjZV_
uVih3r1kuCtks5sMzUQgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314379331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r6wa7NpeV9RS9gBHk4OjSJR7W-t5ks5sMzjtgaJpZM4OL-VK
.

Yes, sorry I was a bit slow there!
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:59
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
I dont believe it is an error message. That is just a response that the API
sends back. Yours looks like it was a success and accepted. if you remove
or comment out the following lines, you should not get any response other
than your email being sent:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();
On Tue, Jul 11, 2017 at 1:57 AM, ianh2<EMAIL_ADDRESS>wrote:

Interesting. I get the error message but the mail still gets sent. I
wasn't expecting that
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:41
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
Awesome. Glad I could help out a little bit.
On Tue, Jul 11, 2017 at 1:39 AM, ianh2<EMAIL_ADDRESS>wrote:

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date:
Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain;
charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6]
=>
X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] =>
Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] =>
Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers:
Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] =>
Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason:
https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14]
=> )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I ran into the same issue. I think that issue with the verification is
due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to
V3
(#412)
I was wondering if there was any update on this. I myself am having
this
same issue, where "Class 'SendGrid\Client' not found". All I have is
the
base download of the sendgrid-php codebase and I have changed the
require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe- https://github.com/notifications/unsubscribe-
auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412#  <https://github.com/sendgrid/sendgrid-php/issues/412# <> <
https://github.com/sendgrid/sendgrid-php/issues/412# >
issuecomment-314370150>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/ < <
https://github.com/notifications/unsubscribe-auth/>
ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314371168>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6kuMy_
nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412# <https://github.com/sendgrid/sendgrid-php/issues/412#>
issuecomment-314374781>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_ https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_
gyfN1GtaaWNSks5sMzSpgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314375239>
, or mute the thread <https://github.com/notifications/unsubscribe- https://github.com/notifications/unsubscribe-
auth/AceR6kXIpEWxhPBBX9gv-FiXNp74LqIZks5sMzUQgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6tNQEbMwweSApjZV_
uVih3r1kuCtks5sMzUQgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314379331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r6wa7NpeV9RS9gBHk4OjSJR7W-t5ks5sMzjtgaJpZM4OL-VK
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314379881 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6im8l89CBN8J6NP5M0fc1m5JSyb0ks5sMzlsgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6m-xKCnxhWRzQwg6w7ATvvqA9LiNks5sMzlsgaJpZM4OL-VK.gif

What I've done to get it to work is

I have a folder at the same level as public_html where I store my database connection data

I created a new file in there that declares a  $APIKey variable

I included a require path to the folder and file

I removed the getenv() code

This works

I do get an additional "=" character at the end of the email which I also get with the V2 API. This is foxing me as I can't see where it comes from
The good thing is that I'm accessing the API via this
require ($topublic."sendgrid-php3/sendgrid-php.php");
This appears to allow me to have both APIs (V2 and V3) co existing and will allow me to migrate my code gracefully
What I need now is for Elmer to tell us how to include an attachment using the library
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:59
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
I dont believe it is an error message. That is just a response that the API
sends back. Yours looks like it was a success and accepted. if you remove
or comment out the following lines, you should not get any response other
than your email being sent:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();
On Tue, Jul 11, 2017 at 1:57 AM, ianh2<EMAIL_ADDRESS>wrote:

Interesting. I get the error message but the mail still gets sent. I
wasn't expecting that
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:41
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
Awesome. Glad I could help out a little bit.
On Tue, Jul 11, 2017 at 1:39 AM, ianh2<EMAIL_ADDRESS>wrote:

If I remove the getenv() around the API key I get
202Array ( [0] => HTTP/1.1 202 Accepted [1] => Server: nginx [2] => Date:
Tue, 11 Jul 2017 08:37:48 GMT [3] => Content-Type: text/plain;
charset=utf-8 [4] => Content-Length: 0 [5] => Connection: keep-alive [6]
=>
X-Message-Id: s84O77EqTEKAl8LLztwx-Q [7] => X-Frame-Options: DENY [8] =>
Access-Control-Allow-Origin: https://sendgrid.api-docs.io [9] =>
Access-Control-Allow-Methods: POST [10] => Access-Control-Allow-Headers:
Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [11] =>
Access-Control-Max-Age: 600 [12] => X-No-CORS-Reason:
https://sendgrid.com/docs/Classroom/Basics/API/cors.html [13] => [14]
=> )
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 09:24
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3
(#412)
I ran into the same issue. I think that issue with the verification is
due
to how you are setting your API key. if you remove the setenv() function
and just hardcode the API key, it should work. But it is not a best
practice to hardcode API keys into your codebase.
On Tue, Jul 11, 2017 at 1:19 AM, ianh2<EMAIL_ADDRESS>wrote:

Hi Barney
Yes Elmer seems to have fixed it although I am getting a verification
error that seems to be associated with the API key
Ian
From: Barneybarker<EMAIL_ADDRESS>Sent: 11 July 2017 08:38
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to
V3
(#412)
I was wondering if there was any update on this. I myself am having
this
same issue, where "Class 'SendGrid\Client' not found". All I have is
the
base download of the sendgrid-php codebase and I have changed the
require
statement to: require("sendgrid-php/lib/loader.php");.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314361038>
, or mute the thread <https://github.com/notifications/unsubscribe- https://github.com/notifications/unsubscribe-
auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6tNUHBHc3Uv3IOPQXxfumW8x8xdFks5sMyZlgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6rlh9wZZhQdfOkiw8rHz0tCfCg
awks5sMyZlgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412#  <https://github.com/sendgrid/sendgrid-php/issues/412# <> <
https://github.com/sendgrid/sendgrid-php/issues/412# >
issuecomment-314370150>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/ < <
https://github.com/notifications/unsubscribe-auth/>
ARM2r3GuvHDQNImVmbWzRUn9gwKdTQ2Wks5sMzAvgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#
issuecomment-314371168>
, or mute the thread <https://github.com/notifications/unsubscribe-auth/ https://github.com/notifications/unsubscribe-auth/
https://github.com/notifications/unsubscribe-auth/
AceR6mcNWlJkBFv38Nd9KucG0BGPoXYoks5sMzE6gaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6kuMy_
nKbGoxy9pMi9zIDNTxtWKGks5sMzE6gaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/sendgrid/sendgrid-php/issues/412# <https://github.com/sendgrid/sendgrid-php/issues/412#>
issuecomment-314374781>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_ https://github.com/notifications/unsubscribe-auth/ARM2rxoDECaFj7SAcD9_
gyfN1GtaaWNSks5sMzSpgaJpZM4OL-VK>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314375239>
, or mute the thread <https://github.com/notifications/unsubscribe- https://github.com/notifications/unsubscribe-
auth/AceR6kXIpEWxhPBBX9gv-FiXNp74LqIZks5sMzUQgaJpZM4OL-VK> . <
https://github.com/notifications/beacon/AceR6tNQEbMwweSApjZV_
uVih3r1kuCtks5sMzUQgaJpZM4OL-VK.gif>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314379331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARM2r6wa7NpeV9RS9gBHk4OjSJR7W-t5ks5sMzjtgaJpZM4OL-VK
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314379881 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6im8l89CBN8J6NP5M0fc1m5JSyb0ks5sMzlsgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6m-xKCnxhWRzQwg6w7ATvvqA9LiNks5sMzlsgaJpZM4OL-VK.gif

Has this been solved yet or any clear instructions? This is pretty crazy just to be able to use the API - I had all of these problems skimming through fiddling for a few hours to try and make it work.
If anything, please just link me to a working version with simple instructions.

Hi Joe
I think it's been solved but there aren't clear instructions. The example is incorrect and we need guidance on how to access the API key as the example will give an error
I think it's disappointed that there aren't 1) Clear, workable examples & 2) A migration path from V2 to V3
Ian
From: JoeD101<EMAIL_ADDRESS>Sent: 11 July 2017 12:44
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Has this been solved yet or any clear instructions? This is pretty crazy just to be able to use the API - I had all of these problems skimming through fiddling for a few hours to try and make it work.
If anything, please just link me to a working version with simple instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314418263 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6rHsWVJ9KPzAAnd82JJvtyYtvkwvks5sM2AUgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6kLbilWybGb0oSKdW7HHe1xP6ti2ks5sM2AUgaJpZM4OL-VK.gif

Hello @JoeD101,
My apologies. I have a build script that is broken which I will be repairing so that those not using composer won't have to go through all this trouble.
Please use these files: sendgrid-php.zip and the documentation included in this repo.
If you continue to have trouble with the file I've included, please reach back out.
This library is about to be upgraded shortly and I will keep the comments in this thread as reference to help ease usage. Thank you so much for the valuable feedback!
With Best Regards,
Elmer

@thinkingserious Thank you for your response. I got that package to work just fine with V3 API code for sending emails.

Sweet, thanks for the follow up!

Things that need to be documented

Sending an e-mail to more than one person

Repeating $to seems to work
$to = new SendGrid\Email(null<EMAIL_ADDRESS><EMAIL_ADDRESS>");
$to = new SendGrid\Email(null<EMAIL_ADDRESS><EMAIL_ADDRESS>");

Adding a cc

Adding $cc doesn't seem to work (either that or the second "to" didn't work)
$cc = new SendGrid\Email(null, "<EMAIL_ADDRESS>test@example.com");

How do you add an attachment?

Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 12 July 2017 02:57
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Sweet, thanks for the follow up!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314622227 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6ioss7Yrw5QYY64QdGbrVDwGfK5bks5sNCfrgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6hXv5XYtygRUw93Wux4M8e9gIT-Kks5sNCfrgaJpZM4OL-VK.gif

Hi @ianh2,
Thanks again for taking the time to give us some feedback!

Here is a complete example that you can use to execute various use cases, including the ones you mentioned.

Please take a look at our upcoming refactor here and let me know what you think. Thanks!

With Best Regards,
Elmer

Hi Elmer
The method of multiple addressees seems clumsy compared with the V2 interface. I wondering how to do it if I have a variable number of addresses. Could I use an array?
The puzzler is this line
$attachment->setContent("TG9yZW0gaXBzdW0gZG9sb3Igc2l0IGFtZXQsIGNvbnNlY3RldHVyIGFkaXBpc2NpbmcgZWxpdC4gQ3JhcyBwdW12");
Is this fixed (I hope so!)?
On the refactor page, if you are dealing with idiots like me who blindly copy the code, the line
$apiKey = getenv('SENDGRID_API_KEY');
Will not work and will give an authentication error.
The below is good and answers my first question
$tos = [
new<EMAIL_ADDRESS>"Example User1"),
new<EMAIL_ADDRESS>"Example User2"),
new<EMAIL_ADDRESS>"Example User3")
];
Much better examples!
I've got a major paragliding competition this weekend in Grasmere so can't do much more until next week
Thanks for your help
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 12 July 2017 20:22
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hi @ianh2 https://github.com/ianh2 ,
Thanks again for taking the time to give us some feedback!

 Here is a complete example <https://github.com/sendgrid/sendgrid-php/blob/master/examples/helpers/mail/example.php>  that you can use to execute various use cases, including the ones you mentioned.

 Please take a look at our upcoming refactor here <https://github.com/sendgrid/sendgrid-php/issues/413>  and let me know what you think. Thanks!

With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-314870822 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6tpET4jUUzROWJ-vB-Jyz0SXE4A-ks5sNRzzgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6sDkeHx_tyrO99CsQ6otjWarJHuIks5sNRzzgaJpZM4OL-VK.gif

Hi @ianh2,
Great feedback once again!
Good luck and have fun with the paragliding competition, that sounds amazing!
Unfortunately, we don't have an addTos() function to accept an array today, but this is coming with the new version.
This line $apiKey = getenv('SENDGRID_API_KEY'); is key for us, since we are trying to keep people from putting their keys in their source code. I'll make sure to clarify this.
With Best Regards,
Elmer

Elmer
I've used your example "function helloEmail()" as set out in the example. It runs without errors but no mail get sent.
I have to say that I am very bemused by this upgrade. The SendGrid web site says "SendGrid provides libraries to help you quickly and easily integrate with the v3 Web API". With V2 you had a simple and straightforward interface. With V3 you seem to have gone out of your way to make it more complex and less easy to implement. I'm not certain what library I'm supposed to using and it does look like you're writing a completely new one. There doesn't seem to have been much thought on updating from V2 to V3 and easing that transition.
I think the best way forward is for me to stick to V2 and wait for something stable to emerge.
The V3 is real step backwards; just consider sending an attachment
V2
$email->setAttachment($uploadfile);
V3
$attachment2 = new Attachment();
$attachment2->setContent("BwdW");
$attachment2->setType("image/png");
$attachment2->setFilename("banner.png");
$attachment2->setDisposition("inline");
$attachment2->setContentId("Banner");
$mail->addAttachment($attachment2);

7 lines of code instead of 1!

Hello @ianh2,
What was the error message? Also, I'd be happy to look at your request object to help debug. Just drop in this line of code before you send the email: echo json_encode($mail, JSON_PRETTY_PRINT), "\n";
The new helper will allow you to do this:
$msg->addAttachment("balance_001.pdf",
                    "base64 encoded content",
                    "application/pdf",
                    "attachment",
                    "Balance Sheet");

As for the step backward, this is true. The PHP SDK that supported v2 was well done, but it was tightly coupled to the v2 mail/send endpoint, as was our other 6 SDKs. So we took a step back to refactor that coupling (and removing third party dependencies while the hood was up, in this SDKs case, it was Guzzle) and allow for support of all of our 233 endpoints.
The final step of this project, which has unfortunately taken far too long, is reflected in this issue. Given your PHP expertise and usage of SendGrid, your feedback would be tremendous. Please do take a look when you get a chance. We want to release those helpers ASAP.
With Best Regards,
Elmer

Hi Elmer
I think I said in the e-mail that I didn't get any errors but didn't get an e-mail either
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 19 July 2017 20:12
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Hello @ianh2 https://github.com/ianh2 ,
What was the error message https://github.com/sendgrid/sendgrid-php/blob/master/TROUBLESHOOTING.md#error-messages ? Also, I'd be happy to look at your request object to help debug. Just drop in this line of code before you send the email: echo json_encode($mail, JSON_PRETTY_PRINT), "\n";
The new helper will allow you to do this:
$msg->addAttachment("balance_001.pdf",
"base64 encoded content",
"application/pdf",
"attachment",
"Balance Sheet");
As for the step backward, this is true. The PHP SDK that supported v2 was well done, but it was tightly coupled to the v2 mail/send endpoint, as was our other 6 SDKs. So we took a step back to refactor that coupling (and removing third party dependencies while the hood was up, in this SDKs case, it was Guzzle) and allow for support of all of our 233 endpoints https://sendgrid.com/blog/stoplight-io-to-test-api-endpoints/ .
The final step of this project, which has unfortunately taken far too long, is reflected in this issue https://github.com/sendgrid/sendgrid-php/issues/413 . Given your PHP expertise and usage of SendGrid, your feedback would be tremendous. Please do take a look when you get a chance. We want to release those helpers ASAP.
With Best Regards,
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316487627 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6senSDKTfnABmb4lvAWond9sdDPBks5sPlUQgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6kh9uIaGg0Qi8yscRvzIGCN4AVaVks5sPlUQgaJpZM4OL-VK.gif

HI @ianh2,
If you do this, you should be able to grab the error message.

Hi Elmer
That’s not the same code that you use in your example
The example says
$response = $sg->client->mail()->send()->post($request_body);
Your link says
$response = $sendgrid->client->mail()->send()->post($mail);

If I use the latter I get
[Wed Jul 19 20:53:18 2017] [error] [client <IP_ADDRESS>:35200] AH01215: PHP Fatal error:  Call to a member function mail() on a non-object in /home/sites/...../public_html/testarea/CSC_TestMail.php on line 112: /home/sites/.../public_html/testarea/CSC_TestMail.php
And the page does not complete
If I use the former then the page completes to the footer. I get no error message and no emails
Signing off until tomorrow
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 19 July 2017 20:46
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
HI @ianh2 https://github.com/ianh2 ,
If you do this https://github.com/sendgrid/sendgrid-php/blob/master/TROUBLESHOOTING.md#error-messages , you should be able to grab the error message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316495938 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6pteQypTlthrQ4jJCs0BGIYPZAx2ks5sPlz4gaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6gvVmhRCXhNTWK_SvZTYxhvQ4OL8ks5sPlz4gaJpZM4OL-VK.gif

@ianh2,
Please ping me when you start working on this again tomorrow so I can try to help you in real time.
If you start before me, can you tell me these values after you send the email:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();

401Array ( [0] => HTTP/1.1 401 Unauthorized [1] => Server: nginx [2] => Date: Thu, 20 Jul 2017 11:41:49 GMT [3] => Content-Type: application/json [4] => Content-Length: 88 [5] => Connection: keep-alive [6] => X-Frame-Options: DENY [7] => Access-Control-Allow-Origin: https://sendgrid.api-docs.io [8] => Access-Control-Allow-Methods: POST [9] => Access-Control-Allow-Headers: Authorization, Content-Type, On-behalf-of, x-sg-elas-acl [10] => Access-Control-Max-Age: 600 [11] => X-No-CORS-Reason: https://sendgrid.com/docs/Classroom/Basics/API/cors.html [12] => [13] => ) {"errors":[{"message":"Permission denied, wrong credentials","field":null,"help":null}]}
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 20 July 2017 00:34
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
@ianh2 https://github.com/ianh2 ,
Please ping me when you start working on this again tomorrow so I can try to help you in real time.
If you start before me, can you tell me these values after you send the email:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316550997 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6m1g7-IontbCUyzDA5jynRaWl17-ks5sPpJKgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6hvhzrbZlJHpY4LnIh2m5bP0zqArks5sPpJKgaJpZM4OL-VK.gif

Puzzling because I've got the APIKey as a system variable and this worked before
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 20 July 2017 00:34
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
@ianh2 https://github.com/ianh2 ,
Please ping me when you start working on this again tomorrow so I can try to help you in real time.
If you start before me, can you tell me these values after you send the email:
echo $response->statusCode();
print_r($response->headers());
echo $response->body();
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316550997 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6m1g7-IontbCUyzDA5jynRaWl17-ks5sPpJKgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6hvhzrbZlJHpY4LnIh2m5bP0zqArks5sPpJKgaJpZM4OL-VK.gif

What happens when you set the APIKey directly, like so? https://github.com/sendgrid/sendgrid-php/blob/master/TROUBLESHOOTING.md#environment-variables-and-your-sendgrid-api-key

If I enter the API key directly, then I get 2 e-mails. However both are to the same person. It looks like the address in the addto is ignored
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 20 July 2017 16:50
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
What happens when you set the APIKey directly, like so? https://github.com/sendgrid/sendgrid-php/blob/master/TROUBLESHOOTING.md#environment-variables-and-your-sendgrid-api-key
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316746386 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6twnkCSIOfr1uHljahDKIp15uMYEks5sP3dAgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6tvw_r_chVLtCdKWxzbnvCgC12Tfks5sP3dAgaJpZM4OL-VK.gif

Right before you make the post, could you tell me the value of:
echo json_encode($mail, JSON_PRETTY_PRINT), "\n";

Sending it to your personal e-mail
Ian
From: Elmer Thomas<EMAIL_ADDRESS>Sent: 20 July 2017 20:36
To: sendgrid/sendgrid-php
Cc: ianh2; Mention
Subject: Re: [sendgrid/sendgrid-php] Updating from SendGrid V2 API to V3 (#412)
Right before you make the post, could you tell me the value of:
echo json_encode($mail, JSON_PRETTY_PRINT), "\n";
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/sendgrid/sendgrid-php/issues/412#issuecomment-316807496 , or mute the thread https://github.com/notifications/unsubscribe-auth/AceR6tgynBo28uvlip6rJxid6oB21IoDks5sP6wHgaJpZM4OL-VK .  https://github.com/notifications/beacon/AceR6qrlPUCZpW1zGEyShG7x1id8CO2Kks5sP6wHgaJpZM4OL-VK.gif


not loading

Cengage companion tab is not loading so i am unable to submit my assignments

Version: 1.89.1
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

/causedByExtension please use your course support channel.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


consensus: Avoid process proposal when we know correct nodes validated it

Resolves #1230

PR checklist

[ ] Tests written/updated
[X] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments

Thanks @hvanz for this. Before approving , do you agree with the third bullet point in the the description of the issue (#1230)?
If yes, what could we do to cover it in the code?

do you agree with the third bullet point in the the description of the issue (https://github.com/cometbft/cometbft/issues/1230)?
If yes, what could we do to cover it in the code?

That's a valid point. I updated the code to cover this case according to what we discussed.

@hvanz please hold on merging as #1235 has this PR as base branch. Thanks!

@hvanz please hold on merging as #1235 has this PR as base branch. Thanks!

We can go ahead and merge this and I will change the target branch of #1235 to main.


rake app:setup undefined method `noecho'

Looks like the rake task is missing a dependency or something:
# rake app:setup

Welcome to reservations! Before using your application, we need to create an
initial superuser account and set some application-wide configurations.
This superuser account can be used later to create admins, import users
and change any configurations that you set from this script. With that
in mind, let's get started!

We need to start by creating a superuser account. Please enter the
following info:

First Name:
Jonathan
Last Name:
Gazeley
Phone #:

Email Address:
<EMAIL_ADDRESS>Password:
rake aborted!
NoMethodError: undefined method `noecho' for #<IO:<STDIN>>
/usr/src/reservations/lib/tasks/setup_application.rake:45:in `block (2 levels) in <top (required)>'
/usr/local/bundle/gems/rake-12.0.0/exe/rake:27:in `<top (required)>'
Tasks: TOP => app:setup
(See full trace by running task with --trace)

I'm a n00b at Ruby but a quick Google turned up some similar issues (https://github.com/veelenga/jira_report/issues/5 and https://github.com/kentaroi/sshkit-sudo/issues/4) seems like it's just a missing require 'io/console'.

Thank you for the report. We have not, however, been able to reproduce this error. Please use the latest version from https://gitlab.com/yale-sdmp/reservations and report any issues there going forward.


Ambush implementation request

Request to add ambush as an hasAbilityActive that sets all moves to crit.

well i can do that, but note that doing shift + z active all right panel crits, i'll add shift + x for the left panel

it should be have been done 👍🏻


Error on confirming edit in mini.files when cwd isn't trimmed

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.files
Description
The issue happens when i'm trying to confirm an edit to a file/folder that is deep in the folder hierarchy - most prominently during re-naming and moving location of files and folders. Creation and deletion seems to work fine.
Specifically for me the issue starts to occur at about five folders deep: root/one/two/three/four/five/. I don't know if the error occurs due to index depth issues or because the root folder gets hidden due to screen size limitation.
Thanks for your attention.
Neovim version
0.10.1
Steps to reproduce

Create a directory ~5-6 layers deep from root: root/one/two/three/four/five/six
Create some files with mini.files in the deepest directory and try renaming them, the error should occur.

Alternatively, if it's a screen size problem - make enough directories for the root to not be visible.
Here are the relevant setup files:
init.lua

-- Clone 'mini.nvim' manually in a way that it gets managed by 'mini.deps'
local path_package = vim.fn.stdpath('data') .. '/site/'
local mini_path = path_package .. 'pack/deps/start/mini.nvim'
if not vim.loop.fs_stat(mini_path) then
	vim.cmd('echo "Installing `mini.nvim`" | redraw')
	local clone_cmd = { 'git', 'clone', '--filter=blob:none', 'https://github.com/echasnovski/mini.nvim', mini_path }
	vim.fn.system(clone_cmd)
	vim.cmd('packadd mini.nvim | helptags ALL')
	vim.cmd('echo "Installed `mini.nvim`" | redraw')
end

-- Set up 'mini.deps' (customize to your liking)
require('mini.deps').setup({
	path = {
		package = path_package
	}
})

-- Use 'mini.deps'. `now()` and `later()` are helpers for a safe two-stage
-- startup and are optional.
local add, now, later = MiniDeps.add, MiniDeps.now, MiniDeps.later

-- || SETTINGS || =============================================================
now(function()
	vim.g.mapleader = " "
	vim.g.maplocalleader = " "
	vim.opt.isfname:append '@-@'
	vim.opt.relativenumber = true
	vim.opt.scrolloff = 8
	vim.opt.tabstop = 4
	vim.opt.softtabstop = 4
	vim.opt.shiftwidth = 4
	vim.opt.signcolumn = "yes"
	vim.opt.colorcolumn = "80"
	vim.opt.swapfile = false
	vim.opt.undofile = true
	vim.opt.undodir = os.getenv("HOME") .. "/.vim/undodir"
	vim.opt.listchars = { tab = "  ", trail = " ", nbsp = " " }
end)

now(function() require("plugins.mini-files") end) <-------- MINI FILES -------

-- keymaps
later(function()
	vim.keymap.set("n", "-", MiniFiles.open, { desc = "Open MiniFiles" })
end)

plugins/min-files.lua
require('mini.files').setup({
    -- Customization of shown content
    content = {
      -- Predicate for which file system entries to show
      filter = nil,
      -- What prefix to show to the left of file system entry
      prefix = nil,
      -- In which order to show file system entries
      sort = nil,
    },

    -- Module mappings created only inside explorer.
    -- Use `''` (empty string) to not create one.
    mappings = {
      close       = 'q',
      go_in       = 'l',
      go_in_plus  = 'L',
      go_out      = 'h',
      go_out_plus = 'H',
      reset       = '<BS>',
      reveal_cwd  = '@',
      show_help   = 'g?',
      synchronize = '=',
      trim_left   = '<',
      trim_right  = '>',
    },

    -- General options
    options = {
      -- Whether to delete permanently or move into module-specific trash
      permanent_delete = true,
      -- Whether to use for editing directories
      use_as_default_explorer = true,
    },

    -- Customization of explorer windows
    windows = {
      -- Maximum number of windows to show side by side
      max_number = math.huge,
      -- Whether to show preview of file/directory under cursor
      preview = true,
      -- Width of focused window
      width_focus = 50,
      -- Width of non-focused window
      width_nofocus = 15,
      -- Width of preview window
      width_preview = 80,
    },
})

Expected behavior
Normal file/directory renaming functionality
Actual behavior
The error specifically is:
                                                                                3,17-15       All
CONFIRM FILE SYSTEM ACTIONS

~/projects/dotfiles/nvim/nvim-mini/one/two/three/four/five
  RENAME │ bingo => asdfs.ts

E5108: Error executing lua: ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:1482: Invalid window id: 1064
stack traceback:
        [C]: in function 'nvim_win_get_buf'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:1482: in function 'explorer_refresh'
        ...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:820: in function <...e/nvim/site/pack/deps/start/mini.nvim/lua/mini/files.lua:812>
Press ENTER or type command to continue

which seems to happen in the same function as the error in these issues: #477 #606 .

Thanks for the issue!
Indeed, I can reproduce. This seems to happen when the current branch is too deep to fit all windows including preview. I'll look into it after the next 'mini.nvim' release.

This should finally be fixed on latest main. Thanks again for finding this!


Use CFPB brand colors in cf-buttons component variables

Adds CFPB brand colors as the default variable values for colors in cf-buttons. Our docs show the brand colors already, since they have a cf-theme-overrides.less stylesheet.
Depends on #561 - this includes changes from that branch in the diff.
Testing
In this repo:

check out this branch
run npm run build
run npm run cf-link

In a second local clone of this repo in a different folder:

pull down and check out the gh-pages-rm-overrides branch: git fetch && git checkout gh-pages-rm-overrides
if you haven't already in this second clone, run all the install steps for the project
run npm run cf-link
run npm run build
run npm start
Compare colors on local site:

http://localhost:3000/components/cf-buttons/
to the live cf documentation to make sure they match: https://cfpb.github.io/capital-framework/components/cf-buttons/

Review

@user

Screenshots

Notes

Todos

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Works great! Add a line to the changelog and :shipit: .

Note: The site doesn't build w/ Node 8 without updating require-dir and parallelshell. I switched to Node 6 and it built a-okay.


Build issue with newest rust version, update time dependency?

Hello. Homebrew maintainer here, we noticed a build failure while rebuilding condure:
error[E0282]: type annotations needed for `Box<_>`
  --> /Users/michkapopoff/Library/Caches/Homebrew/cargo_cache/registry/src/index.crates.io-6f17d22bba15001f/time-0.3.22/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
help: consider giving `items` an explicit type, where the placeholders `_` are specified
   |
83 |     let items: Box<_> = format_items
   |              ++++++++

This is fixed with time 0.3.36. Can you please check and update?

Fixed and tagged 1.10.1.


Fix equivalent example in README

The state variable came from nowhere.

Using this library, the above is exactly equivalent to the following code:
import { reducerWithInitialState } from "typescript-fsa-reducers";

const reducer = reducerWithInitialState(INITIAL_STATE)
    .case(setName, (action, name) => ({ ...state, name }))
    .case(addBalance, (action, amount) => ({
        ...state,
        balance: state.balance + amount,
    }))
    .case(setIsFrozen, (action, isFrozen) => ({ ...state, isFrozen }));

You're absolutely right, thank you very much for the PR!


grewhound, amphibiotica

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,g]

/trunk merge


Types added to some JSON RPC errors

To avoid the situations where we are matching string errors with the use of match, indexOf, or includes we can do this only once in JSON RPC provider.
This logic can be deleted when nearcore will start returning typed errors.

@volovyk-s let's make sure we have tests for these


Extra

Apoyo para extraer bienes de engaños

I don't understand sorry - using English would be appreciated

It is that a page called e bay xh shop twisted me


docs: remove npm command extra @latest

Description

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

This is needed to force npm to get tge latest version, the last installed one will be used if not. pnpm create does this by default

When i use the command npm create vite@latest, will be got.

I try use npm create vite, it's worked!

That is expected, npm doesnt need to ask for confirmation to use an old version in your machine

Thank for you answer


Feature/disable select all for feature phones

@Mitso Make sure to load the js file when JS is enabled/supported what is the ideal way of doing this?


Path not detected.

ErsatzTV can't detect if path exist or not/failed to scan media. When I tried to add ffmpeg path it said that path doesn't exist even it exist. Then I just use variable environtment method. Arter that, I add path for local media and it not scanning any media even I put path to only one show without any subfolder.
UPDATE: It detected folder and video file in it when I checked on sqlite file but it not showing in ErsatzTV

Which release are you using, docker or something else?

Which release are you using, docker or something else?

I'm use Windows.

Can you give an example ffmpeg executable path that ErsatzTV says does not exist? Perhaps you forgot to include the ffmpeg.exe at the end?

C:\ffmpeg\bin\ffmpeg.exe

I think the path check happens when the text box loses focus, so you might need to click/tab out of the text box after pasting for it to check again. Scanning any libraries will not work if ffprobe.exe is unavailable.

Scanning any libraries will not work if ffprobe.exe is unavailable.

ffprobe.exe also included. Actually, it detected song on my computer but then it disappeared and not scanning anything.

Can you screenshot the health checks on the ETV homepage?

Are you able to join the discord? It might be easier to continue this conversation there.

I have discord. My username same like I used in here.


[Watch] Not receiving updates when using transaction

I receive updates only if i'm not using transaction()
It works fine without it.
The stream is created outside the transaction and i use await for all the queries inside the transaction.
moor_flutter: ^3.0.0
moor_ffi: ^0.6.0
moor_generator: ^3.0.0

You should only use moor_flutter OR moor_ffi.
See https://moor.simonbinder.eu/docs/getting-started/starting_with_sql/#adding-the-dependency
This probably has no impact on your issue tho.

Can you post more details and possibly some code to reproduce this? Moor has tests to cover stream behavior in transactions, so I'll need more information to track this down.

I receive updates only if i'm not using transaction()
It works fine without it.
The stream is created outside the transaction and i use await for all the queries inside the transaction.
moor_flutter: ^3.0.0
moor_ffi: ^0.6.0
moor_generator: ^3.0.0

Just remember always await every (query,insert,updte,etc) inside your transaction.

@deep34 I can't act on this issue without further information, so I'll close this. If you still run into this problem, please provide more information about what you're doing. I'll re-open the issue in then.


urxvt derivation sets TERMINFO incorrectly

Describe the bug
$ env | grep TERMINFO=
TERMINFO=$terminfo/share/terminfo

... what's $terminfo doing there?
I suppose it doesn't actually cause any problems given that TERMINFO_DIRS is set, but still, it shouldn't be there.
Steps To Reproduce
Steps to reproduce the behavior:

Install urxvt through nixpkgs (pkgs.rxvt-unicode)
Start a terminal (urxvt)
Check your environment variables as above

Cause
The offending line is clearly
https://github.com/NixOS/nixpkgs/blob/88ca7b5b349dc7d285fbc25e0d7099602ab29192/pkgs/applications/terminal-emulators/rxvt-unicode/default.nix#L52
I suspect this may have worked at one point in the past, then stdenv's escaping logic was changed?
Expected behavior
The variable should be set to a directory that actually exists and has the appropriate information in it, or not set at all.
Notify maintainers
@rnhmjoj
Metadata
I doubt it's relevant, but:
$ nix-shell -p nix-info --run "nix-info -m"
these 4 paths will be fetched (0.39 MiB download, 1.89 MiB unpacked):
  /nix/store/7gdji02y6rbnzv5mcxcf3mwksk785y8j-stdenv-linux
  /nix/store/7wy23fyz7lszd7izsla09d5bzmc57x22-gnumake-4.3
  /nix/store/86zys752g7fg2mwad7yhll285y9w3lym-patchelf-0.12
  /nix/store/wzach25vwapl3sa6f7gylqank3jhhsin-bash-interactive-4.4-p23-dev
copying path '/nix/store/wzach25vwapl3sa6f7gylqank3jhhsin-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/7wy23fyz7lszd7izsla09d5bzmc57x22-gnumake-4.3' from 'https://cache.nixos.org'...
copying path '/nix/store/86zys752g7fg2mwad7yhll285y9w3lym-patchelf-0.12' from 'https://cache.nixos.org'...
copying path '/nix/store/7gdji02y6rbnzv5mcxcf3mwksk785y8j-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.52, NixOS, 21.05.20210729.2262d78 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20210707_02dd6bb`
 - nixpkgs: `/etc/nix/path/nixpkgs`

#140172

btw arch has a lot of configure options, i wonder if we should add some of them? https://github.com/archlinux/svntogit-community/blob/130e74f9e6cf104fb56d58173266506068dccb90/trunk/PKGBUILD#L32

btw arch has a lot of configure options, i wonder if we should add some of them?

I'll take a look, but I suspect most of these are enabled by default.

The only options off by default that we don't enable in Nixpkgs are
--enable-assert (default: off)
--enable-smart-resize (default: off)

The first is for only useful when developing urxvt, the description for
the second says:

Add smart growth/shrink behaviour when resizing. This should keep
the window corner which is closest to a corner of the screen in a
fixed position.

I'm not sure how useful this is because I use a tiling WM.
Anyway, the option is off even in Arch Linux due to a bug.


[KARAF-3827] - Fix Cave rest service doesn't retrieve the repositories

Fix the response of REST service, add JSON response object.
The structure of the object have change :

Repository object tag with JAXRS annotations, attribute name, location, increment and a list of Resource

Resource object tag with JAXRS annotations, attribute identity, type, version, artefact link to the XML repository elements.

Add dependency to the org.apache.karaf.util.XmlUtils to parse the XML repository.
Add QueryParam JAXRS annotations on the query param method
Change Produces JAXRS annotations application/xml to appplication/json

R: @jbonofre


⚠ PostgreSQL has degraded performance

In 3328e70, PostgreSQL (https://news.ycombinator.com) experienced degraded performance:

HTTP code: 200
Response time: 2414 ms

Resolved: PostgreSQL performance has improved in 6492907 after 19 minutes.


Support rxjs Observable and Promise as return types for RPC client methods

Right now, using RPC requires working with the types UnaryCall, ServerStreamingCall.
Unary example (proto):
let call = client.makeHat({ inches: 23 });

let headers: RpcMetadata = await call.headers;
let response: Hat = await call.response;
let trailers: RpcMetadata = await call.trailers;
let status: RpcStatus = await call.status;

Server-streaming example (proto):
let call = client.serverStream(AllMethodsRequest.create());

let headers: RpcMetadata = await call.headers;
for await (let msg: AllMethodsResponse of call.response) {
}
let trailers: RpcMetadata = await call.trailers;
let status: RpcStatus = await call.status;

The types are designed to allow for all gRPC features. For example, it is valid for a server to send message(s), then an error status (and the client should always check the status).
In most cases, however, this complexity is not necessary. There are some facilities to make it easier to use for standard use cases:
// waits until the entire call is finished (and status is okay)
let {response} = await client.makeHat({ inches: 23 });

But in the end, this is too complicated. Most of the time, I just want a response, and having to read up on the semantics is not a good DX.
I would like to have an option to have the methods return well-known async types Observable (rxjs) and Promise:

Example for unary method with Promise return type:makeHat(request: Size, options?: RpcOptions): Promise<Hat>

Unary with Observable:makeHat(request: Size, options?: RpcOptions): Observable<Hat>

Server-streaming with Promise:makeManyHats(request: Size, options?: RpcOptions): AsyncIterable<Hat>

Server-streaming with Observable:makeManyHats(request: Size, options?: RpcOptions): Observable<Hat>

#7 paves the way for a new service option:
service Haberdasher {
    // change the method signatures of this service client to use rxjs Observable:
    (ts.rpc_style) = RXJS;

    rpc MakeHat (Size) returns (Hat);
}

There should probably also be a plugin parameter to change the default the style.
Still pondering whether it should be possible to generate multiple styles in one go...

The new feature has been merged.


Updated links in Code of Conduct

Final link destinations.

Hi @timGerken, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Tim Gerken). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Feature request: Bluetooth audio streaming

Hi,
is it possibly to add Bluetooth audio streaming? Could also be as a preset station to save buttons.
Here are some examples I found online:
https://www.pschatzmann.ch/home/2020/03/27/1255/
https://github.com/MrBuddyCasino/ESP32_MP3_Decoder

As far as I know, a combination of WiFi and BT simultaneously is not possible.

I meant that you can select if you want to play a Webradio station or receive a Bluetooth audio stream from your smartphone for example. But not at the same time.
But then there is the problem with the webserver and mqtt. Didn't think of that.

You may use a BT receiver and feed the output to the radio amplifier.

Hi, during my websearch I found the following content, which shows that Bluethooth and Wifi should be possible in generell. But a totally different setup would be necessary:
https://github.com/BM45/esp-adf/tree/master/examples/advanced_examples/wifi_bt_ble_coex
A webradio with the adf framework, but bluetooth not implemented (yet):
https://github.com/BM45/iRadioMini


Configure CodeMirror to treat dataviewjs as javascript

Fixes #566

This works, thank you

Can also confirm that this works nicely.

Neat!

hooray


Ensure website uses "Material Design" (capital D)

The material.io website says "Material Design", two words, capitalized.
We may use a combo of lower-case and upper-case across the website. It's worth globally searching through and ensure we're consistent and correct.

This page uses lower-case d: https://material.io/guidelines/#
This page uses upper-case: https://material.io/
I'll file an issue with the material team and get an official ruling.

Confirmed with the Material team. Use capital "Material Design" everywhere.


My awesome project ·

https://blog.hchen.xyz/project/my-awesome-project/
Description of my awesome project.

好的。


Removed base.OnException() call from MVC client

Fixes issues #41
Stops attempting to render a Error page as part of the Bugsnag Client, will just report the exception.
Note that the exception show now not be marked as handled.

Cheers @CodeHex!


State Management Login

User story

As a facility manager
I need a secure and efficient login system
So that my session is managed properly.

Acceptance criteria

[ ] login page is default when opening the page (unless you logged in sometime in the last hour and did not log out in the end)
[ ] username "siemens", password "siemens" work (using hashing with the cryptor)
[ ] after the login you can see everything, open multiple tabs where you are logged in
[ ] the name in the top right is shown
[ ] you can log out by yourself or you are automatically logged out after 1h
[ ] you can not change the URL to just avoid the login

Definition of done (DoD)

[ ] All acceptance criteria were met
[ ] Pipeline is building successfully
[ ] Unit/integration tests are passing
[ ] Code has been reviewed and merged into the primary development branch

DoD general criteria

[ ] Developers agreed to release
[ ] Product owner approved features
[ ] Feature has been fully implemented

I would also like to adjust the login page because right now I think it looks a bit cramped. Also the password forgotten and github login are not functional so I would like to remove them

I would like to make it somewhat similar to the normal siemens login. But of course we need a password field and we don't need all the ones on the bottom. Also the colours will be a little different as they are the siemens-brand-dark/light colours and we only have the classic-dark/light available. So I'll see how close I can get to this:

I changed how the login looks to this, which looks cleaner in my opinion

I also thought about it more and I think it makes more sense to use an email adress instead of username. Like that we actually have an email adress to show for the #229 issue. The login works for any of the following email adresses using password "siemens":
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>I took them from the P1 Planning document and added a few more

Btw if you want it to work you have to add the XD_SECRET_KEY to the .env file so you can ask me for it


Admin UI - Download No longer Works When Served from Different URL than the API

Bug Description
On the Admin UI, when a user goes to download the CSV, they are unable to and the server responds with a 404. This is because the request is made from the host that is serving the Admin UI, not the API web server.
The other API calls are working and using the correct URL, but this one isn't. I believe this is caused by RequestCSVDownload, https://github.com/ethyca/fidesops/blob/04f9d70dc36cf08479711aea2551eed4a07ae716/clients/admin-ui/src/features/privacy-requests/privacy-requests.slice.ts#L117, where BASE_API_URN should instead be BASE_URL, so it's an absolute path to the correct server instead of a relative path to the Admin UI server.
Steps to Reproduce

Set up Fidesops (tested on 1.6.0)

I did this by using Fides demo and changing the docker tag to 1.6.1 and changing the .env file to have NEXT_PUBLIC_FIDESOPS_API=http://localhost:8080

Browse to Admin UI and try to download the csv

Expected behavior
CSV should download without error.
Screenshots

Environment

Version: 1.6.0
OS:
Python Version:
Docker Version:

This might actually be working as expected, I don't have it configured to use the new version where the Admin UI is served from the same port as the web server.

Closing because this looks like it's a misconfiguration issue.


Where to find LSH Sampling

In your paper, ref "Table 5: Samplers in RecStudio", you mentioned that LSH based samplers have been implemented. But I cannot find them in your code.

The LSH sampler is in beta version now and can be found in PR #71.


feat: display hardcoded miner upgrade notification

Hardcoded message for miners to upgrade to <IP_ADDRESS>.1

:tada: This PR is included in version 1.55.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Swift iOS 9/Xcode 7 support?

Hello,
when it is expected to have support for iOS 9 / xcode 7?
Thanks for the great library!

can you descript what is the issue? i'm using nmrangeslider with ios 9 and it works.

@dermaaarkus are you using swift?  The issue is that the lack of bundle is causing no images to show up which means you cant see anything with the slider

This is a duplicate of #58 and is solved by #54

Can this ticket be closed? Thanks :)


GFM version 6: what's coming up?

[x] removal of gfm:image altogether, the STB truetype will return into dplug:gui and the loadImage functions, well, aren't that incredible.

[x] will remove rationals from gfm:integers since this package exist now: http://code.dlang.org/packages/rationald

[x] fully nothrow @nogc gfm.core.queue and removal of LockedQueue towards dplug:coreto make it @nogc (@nogc sync primitives cannot be made public domain because based on druntime's code)

No need to make gfm.core.queue @nogc as it's templated.


missing space

Description
missing space
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
326db8d2-b9d4-4634-9c19-a7c835233eff
Paper URL
https://arxiv.org/html/2312.10997v5
Browser
Chrome/<IP_ADDRESS>
Device Type
mac

issue occurs in pdf. marking as resolved


[Feature Request] Add context menu for Python Interactive window

Adding a context menu that hosts different operations allowed in the interactive window would be a great feature. for example restarting, interrupting a kernel, adding a new cell, removing a cell, expanding/collapsing and more importantly copying a cell content or portions of it, can be provided in a context menu.
Currently, the only way we can copy an error or output from the python interactive window is by first making a selection and then pressing Ctrl+C. this is not desirable and an uncalled-for hassle. nearly always after making a selection, I uncontrollably go for a right click to select copy! and bang there is no context menu and I have to reselect what I selected before again!
It would be much appreciated if you could add a context menu where we can select a word, line, or the whole output of a cell, and then copy it.
being able to restart/interrupt a kernel, etc from a context menu is also desirable and greatly appreciated.

Context menus are now supported in the Python interactive window.


KeyError: 'choices'

I seem to get KeyError: 'choices' a lot

Hi @darthludious Can you provide a log or an example of how to reproduce this? It might be related to the LLM model inference output.

Same error here...
Running assistant...
Prompt: edge browser go to YouTube and browse Tesla videos
Called from: assistant
Listening...
Listening...
Listening...
AI selected application: Microsoft Edge
AI Analyzing: Microsoft Text Input Application
Generating a test case with the assistant. Image visioning started. Analyzing the application Microsoft Text Input Application for context.
Exception in thread Thread-5 (run_assistant):
Traceback (most recent call last):
File "C:\Users<name>\AppData\Local\miniconda3\envs\llm2\Lib\threading.py", line 1038, in _bootstrap_inner
self.run()
File "C:\Users<name>\AppData\Local\miniconda3\envs\llm2\Lib\threading.py", line 975, in run
self._target(*self._args, **self._kwargs)
File "C:\Code\pywinassistant\pywinassistant-main\core\assistant.py", line 164, in run_assistant
assistant(assistant_goal=action, called_from="assistant")
File "C:\Code\pywinassistant\pywinassistant-main\core\driver.py", line 253, in assistant
assistant_goal = imaging(window_title=app_name, additional_context=additional_context, screenshot_size='Full screen')['choices'][0]['message']['content']
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^
KeyError: 'choices'

AI selected application: Spotify
AI Analyzing: Spotify
Generating a test case with the assistant. Image visioning started. Analyzing the application Spotify for context.
Listening...
Exception in thread Thread-10 (run_assistant):
Traceback (most recent call last):
File "C:\Users\Goose\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 1052, in _bootstrap_inner
self.run()
File "C:\Users\Goose\AppData\Local\Programs\Python\Python312\Lib\threading.py", line 989, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\Goose\Desktop\jarvis\core\assistant.py", line 164, in run_assistant
assistant(assistant_goal=action, called_from="assistant")
File "C:\Users\Goose\Desktop\jarvis\core\driver.py", line 253, in assistant
assistant_goal = imaging(window_title=app_name, additional_context=additional_context, screenshot_size='Full screen')['choices'][0]['message']['content']
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^
KeyError: 'choices'
Listening...

The fix is to add the API keys correctly:
add your API Key in /core/core_api.py  ->  line 3: client = OpenAI(api_key='insert_your_api_key_here')
add your API Key in /core/core_imaging.py  ->  line 12: api_key = 'insert_your_api_key_here'
For more information, you could check the Readme.md for the complete usage and installation guide.
Special thanks to @NadavIs56 !

Ahhhhhh the core_imaging.py - missed that one thank you


Research hiding votes until curation expires

commit-reveal
obfuscation
homomorphic encryption
zk-SNARKS
multi-signature schemes

Another option: https://scrt.network/


Remove private_endpoint_attachment resource

See https://github.com/ClickHouse/terraform-provider-clickhouse/issues/71#issuecomment-2271549956
The private_endpoint_attachment fights with the private_endpoint_ids field of the clickhouse_service resource. They can never work together so I am going to remove the recently added private_endpoint_attachment resource.
I also updated the private link sample to make it work

CC @egor-krv as you added this resource recently


feat(odd): menuList and MenuItem ODD support

closes RAUT-403
Overview
Change MenuList and MenuItem to be used as the overflow menu in ODD
https://user-images.githubusercontent.com/66035149/232887525-238b58ce-42c1-4c15-a920-b6206d3f6998.mov
Test Plan
Review the storybooks for MenuList and MenuItem and review the menu buttons in the desktop app to make sure they retain correct UI and review the Navigation menu in the ODD to make sure it has correct Ui.
Changelog

Change MenuItem to take in isAlert prop and use media query, modify storybook and test
Change MenuList to take in isOnDevice and OnClick props, change button prop to be children and fix all usages of it in the desktop (it was only used in Module overflow menu). Modify storybook and test
Change NavigationMenu to use MenuItem and MenuList
add ODD viewport to storybook/preview

Review requests

see test plan

Risk assessment
low

The styling on the ODD does not match the design: https://www.figma.com/file/AoTLAYuWawlaWItB1umOjr/Release%3A-Opentrons-Flex-Touchscreen-Designs?node-id=8292-140777&t=JBQTG6cB56MTBzOz-0

Maybe I had something cached somewhere. Things are looking better for me now.

lgtm! Can you make a ticket to convert app/src/pages/OnDeviceDisplay/ProtocolDashboard/LongPressModal.tsx to use these components?

@ewagoner, i went ahead and changed LongPressModal to use these since it is a small lift 😄