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

docs: document order of execution of middlewares #102

Merged
merged 1 commit into from
Oct 8, 2024
Merged

docs: document order of execution of middlewares #102

merged 1 commit into from
Oct 8, 2024

Conversation

cyberhck
Copy link
Contributor

@cyberhck cyberhck commented Oct 8, 2024 •
edited by jai
Loading

Pull Request Submission Checklist

Please confirm that you have done the following before requesting reviews:

  • I have confirmed that the PR type is appropriate for the change I am making according to
    the Honest Pull Request and Commit Message Naming Conventions.
  • I have typed an adequate description that explains why I am making this change.
  • I have installed and run standard pre-commit hooks that lints and validates my code.

Description

  • This PR adds a documentation page after introduction that highlights the order of execution.

closes #101

This change is Reviewable

@cyberhck cyberhck requested review from a team as code owners October 8, 2024 03:11
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@su-astika su-astika merged commit 89dd52f into main Oct 8, 2024
10 of 11 checks passed
@su-astika su-astika deleted the nishchal/docs-add-order-of-execution-documentation branch October 8, 2024 03:57
@honestbank-bot
Copy link
Contributor

🎉 This PR is included in version 1.15.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kelu-diao kelu-diao kelu-diao approved these changes

@su-astika su-astika su-astika approved these changes

@taras-zak taras-zak Awaiting requested review from taras-zak taras-zak is a code owner automatically assigned from honestbank/backend-engineers

@peterhonest peterhonest Awaiting requested review from peterhonest peterhonest is a code owner automatically assigned from honestbank/honestbank-engineers

Assignees
No one assigned
Labels
hacktoberfest-accepted released size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

document order of middleware execution
4 participants
@cyberhck @honestbank-bot @kelu-diao @su-astika