Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update vendor yaegashi/msgraph.go to v0.1.2 #1036

Merged
merged 2 commits into from
Mar 21, 2020

Conversation

qaisjp
Copy link
Collaborator

@qaisjp qaisjp commented Mar 15, 2020

They fixed the macOS bug I was having - yaegashi/msgraph.go#11

@qaisjp qaisjp added the bug label Mar 15, 2020
@42wim
Copy link
Owner

42wim commented Mar 15, 2020

Did you rebase / tested the changes in my fork?
Specifically matterbridge/msgraph.go@ce5df2f

@qaisjp
Copy link
Collaborator Author

qaisjp commented Mar 15, 2020

matterbridge binary builds, but I haven't tested ms teams.

The only content commit in your fork is matterbridge/msgraph.go@ce5df2f and I can confirm that the enums are corrected in the v0.1.2

➜ ag "type BodyType"
vendor/github.com/yaegashi/msgraph.go/beta/EnumBody.go
6:type BodyType string

➜ ag "type ChatMessageImportance"
vendor/github.com/yaegashi/msgraph.go/beta/EnumChat.go
6:type ChatMessageImportance string

➜ ag "type ChatMessageType"
vendor/github.com/yaegashi/msgraph.go/beta/EnumChat.go
124:type ChatMessageType string

So I've switched it over from your fork to the upstream repo.

@qaisjp
Copy link
Collaborator Author

qaisjp commented Mar 15, 2020

(The enum types were fixed in yaegashi/msgraph.go@a886f8c upstream)

@42wim
Copy link
Owner

42wim commented Mar 15, 2020

Ok, I'll test this if I can find some time.
Was planning on releasing v1.17.0 soon with the ms teams support.
It's possible that this PR will be merged after v1.17.0 depending on my time :)

@qaisjp
Copy link
Collaborator Author

qaisjp commented Mar 15, 2020

Okay!

@42wim 42wim added this to the v1.17.1 milestone Mar 21, 2020
@42wim 42wim merged commit 76e5fe5 into 42wim:master Mar 21, 2020
@42wim
Copy link
Owner

42wim commented Mar 21, 2020

Thanks! tested and working

@qaisjp qaisjp deleted the feature/upgrade-msteams branch March 21, 2020 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants