Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat: support customized table#168

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
JalinWang wants to merge1 commit intocasbin:masterfromJalinWang:patch-1

Conversation

@JalinWang
Copy link
Member

fix#167

@casbin-bot
Copy link
Member

@tangyang9464 @closetool@sagilio please review

@hsluoyz
Copy link
Member

@basimsahaf@tangyang9464 plz review

@hsluoyz
Copy link
Member

@JalinWang

JalinWang reacted with eyes emoji

@JalinWangJalinWang changed the titlefix: support customized tablefeat: support customized tableJun 27, 2022
@hsluoyz
Copy link
Member

@basimsahaf plz review

@JalinWangJalinWang mentioned this pull requestAug 23, 2022
@hsluoyz
Copy link
Member

@JalinWang fix conflicts:

image

@JalinWang
Copy link
MemberAuthor

@JalinWang fix conflicts:

image

@hsluoyz Done

@hsluoyz
Copy link
Member

@JalinWang fix:

image

@tangyang9464
Copy link
Member

@JalinWang Does this PR support custom v0-v5？

@JalinWang
Copy link
MemberAuthor

JalinWang commentedAug 26, 2022•
edited
Loading

@JalinWang Does this PR support custom v0-v5？

Currently, no. I'll work on it in days.

@JalinWang
Copy link
MemberAuthor

@JalinWang fix:

image

@hsluoyz SORRRRY but now it's okay. Annoying conflict resolving 😣.

@hsluoyz
Copy link
Member

@JalinWang this PR makes code too complicated just to implement a not-so-important extra functionality, gorm-adapter is a benchmarking example and it should be kept simple. I suggest we putting this code into another adapter repo

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

Reviewers

@kilosonckilosoncAwaiting requested review from kilosonc

@sagiliosagilioAwaiting requested review from sagilio

2 more reviewers

@basimsahafbasimsahafbasimsahaf left review comments

@tangyang9464tangyang9464tangyang9464 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

How do I soft delete entries in casbin_rule table?

5 participants

@JalinWang@casbin-bot@hsluoyz@tangyang9464@basimsahaf
[8]ページ先頭
©2009-2025 Movatter.jp