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

chore(BUX-223): replace current router with gin #468

Merged
merged 18 commits into from
Feb 26, 2024
Merged

chore(BUX-223): replace current router with gin #468

merged 18 commits into from
Feb 26, 2024

Conversation

pawellewandowski98
Copy link
Contributor

Pull Request Checklist

  • 📖 I created my PR using provided : CODE_STANDARDS
  • 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
  • 🏠 I tested my changes locally.
  • ✅ I have provided tests for my changes.
  • 📝 I have used conventional commits.
  • 📗 I have updated any related documentation.
  • 💾 PR was issued based on the Github or Jira issue.

Description

  • Replaced router with gin
  • Moved authorization methods into this repository

!!! NOTE: require other PR in go-paymail and PR in Bux to pass the checks

@pawellewandowski98 pawellewandowski98 self-assigned this Feb 16, 2024
@pawellewandowski98 pawellewandowski98 requested a review from a team as a code owner February 16, 2024 09:24
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 16, 2024 •
edited
Loading

Manual Tests

💚 Manual testing by @wregulski resulted in success.

@mergify mergify bot added the feature Any new significant addition label Feb 16, 2024
@mergify Mergify
Copy link
Contributor

mergify bot commented Feb 16, 2024

Welcome to our open-source project @pawellewandowski98! 💘

chris-4chain
chris-4chain requested changes Feb 22, 2024
View reviewed changes
server/server.go Outdated Show resolved Hide resolved
chris-4chain
chris-4chain reviewed Feb 23, 2024
View reviewed changes
actions/admin/models.go Outdated Show resolved Hide resolved
server/server_test.go Outdated Show resolved Hide resolved
actions/utxos/get.go Show resolved Hide resolved
server/auth/middleware.go Outdated Show resolved Hide resolved
actions/xpubs/update.go Outdated Show resolved Hide resolved
actions/transactions/models.go Outdated Show resolved Hide resolved
actions/transactions/models.go Outdated Show resolved Hide resolved
actions/destinations/models.go Outdated Show resolved Hide resolved
server/auth/middleware.go Outdated Show resolved Hide resolved
server/util.go Outdated Show resolved Hide resolved
@chris-4chain chris-4chain requested review from chris-4chain and removed request for chris-4chain February 23, 2024 06:37
dorzepowski
dorzepowski requested changes Feb 26, 2024
View reviewed changes
server/auth/middleware.go Outdated Show resolved Hide resolved
dorzepowski
dorzepowski requested changes Feb 26, 2024
View reviewed changes
server/auth/middleware.go Outdated Show resolved Hide resolved
@wregulski
Copy link
Contributor

Testing

@wregulski wregulski added the tested PR was tested by a team member label Feb 26, 2024
@wregulski wregulski added tested PR was tested by a team member and removed tested PR was tested by a team member labels Feb 26, 2024
@pawellewandowski98 pawellewandowski98 merged commit cfc6391 into master Feb 26, 2024
9 of 10 checks passed
@pawellewandowski98 pawellewandowski98 deleted the feat-223-replace-router branch February 26, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nazarii-4chain Nazarii-4chain Nazarii-4chain left review comments

@chris-4chain chris-4chain chris-4chain approved these changes

@dorzepowski dorzepowski dorzepowski approved these changes

Assignees

@pawellewandowski98 pawellewandowski98

Labels
feature Any new significant addition tested PR was tested by a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@pawellewandowski98 @wregulski @dorzepowski @Nazarii-4chain @chris-4chain