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

Adjust cookie tutorial #41

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Adjust cookie tutorial #41

merged 1 commit into from
Feb 27, 2024

Conversation

rsaz
Copy link
Member

@rsaz rsaz commented Feb 27, 2024

Pull Request Guidelines

Our guidelines for submitting a pull request.

Before submitting a Pull Request, please make sure you have verified the following:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

PR Type

What kind of change does this PR introduce?

  • Documentation fix
  • Documentation update
  • New Feature
  • Translation:: New language
  • Bugfix
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

Please describe the current behavior that you are modifying, or link to a relevant issue.
Issue Number: N/A

-> Response: Cookies tutorial are reflecting previous API

What is the new behavior?

Describe the new behavior or link to a relevant issue.

-> Response: Cookie is updated to the new AppFactory and App class API

Does this PR introduce a breaking change?

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications below.

-> Response: N/A

Other information

Any other information that is important to this PR.

  • N/A

-> Response:

@rsaz rsaz added the doc update Update of documentation label Feb 27, 2024
@rsaz rsaz self-assigned this Feb 27, 2024
@rsaz rsaz merged commit 376485d into main Feb 27, 2024
1 check passed
@rsaz rsaz deleted the feature/update-cookies-tutorial branch February 27, 2024 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc update Update of documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant