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 AZFunctionApp and AZWebApp enum #89

Closed
wants to merge 1 commit into from

Conversation

af001
Copy link

@af001 af001 commented Nov 27, 2024

Changes:

list-function-apps.go
list-web-apps.go

AZFunctionApp and AZWebapp do not correctly display neo4j entities due to the keyword checks failing; specifically functionapp and app. The kind attribute may contain more than one value separated by a comma.

Observed variations from AppServices:

functionapp,workflowapp
<class 'str'>
app,linux
<class 'str'>
functionapp,linux
<class 'str'>

Copy link

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

@af001 af001 force-pushed the update-logic-web branch 3 times, most recently from a3fb9dd to 7f03bd1 Compare November 27, 2024 17:30
@af001
Copy link
Author

af001 commented Nov 27, 2024

I have read the CLA Document and I hereby sign the CLA

@af001 af001 closed this Nov 27, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 27, 2024
@af001 af001 deleted the update-logic-web branch November 27, 2024 17:49
@af001 af001 restored the update-logic-web branch November 27, 2024 17:53
@af001 af001 deleted the update-logic-web branch November 27, 2024 17:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants