Sharepoint SetXHttpMethod should set data to global variable. Cleanup

[Drakonian]

Summary

Sharepoint SetXHttpMethod should set data to global variable. Cleanup

Work Item(s)

Fixes #666

Fixes AB#532990

[JesperSchulz]

Looks like you've got an unused variable. Let's make CI pass before we involve developers in the review 🙂

[Drakonian]

Looks like you've got an unused variable. Let's make CI pass before we involve developers in the review 🙂

How I can do that? I thought CI process starts only after assigning an internal ticket number

[Drakonian]

@JesperSchulz
I mean, can I run/trigger CI myself to make sure everything is ready? :)

[Drakonian]

Fixed

[JesperSchulz]

Looks like you've got an unused variable. Let's make CI pass before we involve developers in the review 🙂

How I can do that? I thought CI process starts only after assigning an internal ticket number

I can kick off the CI process any time. Will do so again to see if it passes now 🙂

[Drakonian]

Looks like you've got an unused variable. Let's make CI pass before we involve developers in the review 🙂

How I can do that? I thought CI process starts only after assigning an internal ticket number

I can kick off the CI process any time. Will do so again to see if it passes now 🙂

I would of course like to run CI at least locally before send code -)

This error looks like an internal

[JesperSchulz]

Let's try this once more, before I bring in the cavalry...

[JesperSchulz]

@Drakonian, now there actually seem to be real compilation issues (e.g. AL0126 No overload for method 'UpdateListItemMetaDataField' takes 4 arguments.). Could you take another look?

[Drakonian]

@JesperSchulz fixed

[JesperSchulz]

All CI checks have passed. Now we'll review 🚀