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

feat: 实现了权益码相关接口 #31

Merged
merged 3 commits into from
Dec 10, 2024
Merged

feat: 实现了权益码相关接口 #31

merged 3 commits into from
Dec 10, 2024

Conversation

MangoGovo
Copy link
Collaborator

@MangoGovo MangoGovo commented Dec 6, 2024 •
edited
Loading

  • 权益码的增删改查
  • 权益码的搜索与筛选
  • 权益码新增信息的统计

@MangoGovo MangoGovo requested review from cbluebird, SugarMGP and Penryn and removed request for cbluebird and SugarMGP December 6, 2024 10:40
@MangoGovo MangoGovo requested a review from SugarMGP December 6, 2024 10:44
cbluebird
cbluebird reviewed Dec 7, 2024
View reviewed changes
Copy link
Collaborator

@cbluebird cbluebird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

把你的import都格式化一下,这个怎么import排序的这么奇怪

@cbluebird
Copy link
Collaborator

pr有些地方删改没什么 必要,最后合成一个commit,messsage为feat:添加权益码功能就可以了

@SugarMGP
Copy link
Collaborator

SugarMGP commented Dec 7, 2024

把你的import都格式化一下,这个怎么import排序的这么奇怪

现在好像都是gci统一排序的

cbluebird
cbluebird reviewed Dec 7, 2024
View reviewed changes
app/controllers/qrcodeController/delete.go Outdated Show resolved Hide resolved
app/controllers/qrcodeController/update.go Show resolved Hide resolved
app/models/qrcode.go Outdated Show resolved Hide resolved
app/utils/database/paginate.go Outdated Show resolved Hide resolved
@MangoGovo MangoGovo force-pushed the MangoGovo/dev branch 3 times, most recently from dfbf6f7 to cd22e7a Compare December 8, 2024 10:48
@MangoGovo MangoGovo requested a review from cbluebird December 8, 2024 11:13
@MangoGovo MangoGovo requested review from SugarMGP and Penryn and removed request for Penryn, cbluebird and SugarMGP December 8, 2024 11:36
@MangoGovo MangoGovo force-pushed the MangoGovo/dev branch 4 times, most recently from 6f53fc9 to 7903b36 Compare December 9, 2024 11:50
@MangoGovo MangoGovo force-pushed the MangoGovo/dev branch 4 times, most recently from 4208b9d to 239c09b Compare December 10, 2024 02:56
@Penryn Penryn merged commit 29d6d10 into dev Dec 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cbluebird cbluebird cbluebird left review comments

@SugarMGP SugarMGP SugarMGP approved these changes

@Penryn Penryn Awaiting requested review from Penryn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MangoGovo @cbluebird @SugarMGP @Penryn