Skip to content

Feedly: update WEBSITE_RUN_FROM_PACKAGE#12985

Merged
v-dvedak merged 3 commits into
Azure:masterfrom
feedly:update-website-run-from-package
Oct 28, 2025
Merged

Feedly: update WEBSITE_RUN_FROM_PACKAGE#12985
v-dvedak merged 3 commits into
Azure:masterfrom
feedly:update-website-run-from-package

Conversation

@Edouard360
Copy link
Copy Markdown
Contributor

Required items, please complete

Change(s):

  • Corrected zip uri (one liner change)

Reason for Change(s):

Version Updated:

  • No

Testing Completed:

  • Yes

@Edouard360 Edouard360 requested review from a team as code owners October 20, 2025 13:58
@Edouard360
Copy link
Copy Markdown
Contributor Author

Edouard360 commented Oct 20, 2025

Would love to merge this as soon as possible, thank you 🙏

(( Note: The download was failing because https://aka.ms/sentinel-Feedly-functionapp still points to the old github address. I emailed AzureSentinel@microsoft.com to update the redirect, and they instead suggested I use Codeless Connector Framework (CCF). I am a little short on bandwidth for this extra migration and would like the current connector to be functional in the short term, so would love to merge this PR in order to update the WEBSITE_RUN_FROM_PACKAGE directly to the github url of the .zip file - I have seen another Solutions IpInfo doing this as part of #11842 ))

@v-shukore v-shukore self-assigned this Oct 22, 2025
@v-shukore v-shukore added the Solution Solution specialty review needed label Oct 22, 2025
@v-shukore
Copy link
Copy Markdown
Contributor

Hi @Edouard360, could you please update the URL into maintemplate as well as in Zip file. Thanks!

@Edouard360
Copy link
Copy Markdown
Contributor Author

@v-shukore thank you for the review, good catch. I updated it in those places as well 🙏

@v-shukore
Copy link
Copy Markdown
Contributor

Hi @Edouard360, could you please accept the CLA to proceed. Thanks!

@Edouard360
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree [company="Feedly"]

@Edouard360
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company=Feedly

@v-shukore
Copy link
Copy Markdown
Contributor

@microsoft-github-policy-service agree company="Feedly"

1 similar comment
@Edouard360
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Feedly"

@v-shukore
Copy link
Copy Markdown
Contributor

Hi @Edouard360, it seems that after accepting the CLA, the validation check didn't pass. Could you please update this branch with master and commit the update, and then accept the CLA again to help pass the validation? Thanks!

@Edouard360
Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Feedly"

@Edouard360
Copy link
Copy Markdown
Contributor Author

@v-shukore I updated it, I hope it's better now 🙏

@v-dvedak v-dvedak merged commit ffe3494 into Azure:master Oct 28, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Solution Solution specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants