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.

Sign up for GitHub

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

Jump to bottom

Update the tutorial Expose a Function with an APIRule #1062

Merged
merged 4 commits into from
Jul 23, 2024
Merged

Update the tutorial Expose a Function with an APIRule #1062

merged 4 commits into from
Jul 23, 2024

Conversation

nataliasitko
Copy link
Contributor

Description

Changes proposed in this pull request:

  • Update the tutorial Expose a Function with an APIRule to use the no_auth access strategy

Related issue(s)

@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. labels Jul 9, 2024
@nataliasitko nataliasitko added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. and removed cla: yes Indicates the PR's author has signed the CLA. labels Jul 9, 2024
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cla: yes Indicates the PR's author has signed the CLA. labels Jul 9, 2024
@nataliasitko nataliasitko assigned grego952 and unassigned grego952 Jul 10, 2024
@pPrecel pPrecel marked this pull request as draft July 22, 2024 11:45
@kyma-bot kyma-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 22, 2024
@pPrecel pPrecel marked this pull request as ready for review July 22, 2024 11:45
@kyma-bot kyma-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 22, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 23, 2024
@nataliasitko nataliasitko force-pushed the update-apirule-tutorial branch from ac3af1c to 07464b8 Compare July 23, 2024 09:40
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Jul 23, 2024
@kyma-bot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@nataliasitko nataliasitko added lgtm Looks good to me! and removed do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. labels Jul 23, 2024
@kyma-bot kyma-bot merged commit f7f865b into kyma-project:main Jul 23, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@triffer triffer triffer left review comments

@grego952 grego952 grego952 approved these changes

@mmitoraj mmitoraj Awaiting requested review from mmitoraj mmitoraj is a code owner

@NHingerl NHingerl Awaiting requested review from NHingerl NHingerl is a code owner

@IwonaLanger IwonaLanger Awaiting requested review from IwonaLanger IwonaLanger is a code owner

Assignees

@grego952 grego952

Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nataliasitko @kyma-bot @triffer @grego952