models pk instead of models id

[sahama]

Fixes #

• Using pk instead of id.

Description of the Change

Checklist

• PR only contains one change (considered splitting up PR)
• unit-test added
• [] documentation updated
• CHANGELOG.md updated (only for user relevant changes)
• author name in AUTHORS

[n2ygk]

Is this a repeat of #1445?

[sahama]

Is this a repeat of #1445?

No. #1445 refer to custom primary key in user model and this is about custom primary key in django-oauth-toolkit models.

[n2ygk]

Is this a repeat of #1445?

No. #1445 refer to custom primary key in user model and this is about custom primary key in django-oauth-toolkit models.

Ah, I see. Since these changes are closely related, can you please put the #1445 changes in here as well? Thanks.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.28%. Comparing base (102c851) to head (16cc0d8).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1446      +/-   ##
==========================================
- Coverage   97.42%   97.28%   -0.15%     
==========================================
  Files          32       32              
  Lines        2096     2096              
==========================================
- Hits         2042     2039       -3     
- Misses         54       57       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[n2ygk]

Thanks for this! Please take a look at the one question I have and resolve that.

[sahama]

Is this a repeat of #1445?

No. #1445 refer to custom primary key in user model and this is about custom primary key in django-oauth-toolkit models.

Ah, I see. Since these changes are closely related, can you please put the #1445 changes in here as well? Thanks.

ok. I get new branch from that. so change are here too.