Skip to content

Conversation

@segiddins
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Apr 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.09%. Comparing base (c999fd0) to head (a24f43e).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4610   +/-   ##
=======================================
  Coverage   97.09%   97.09%           
=======================================
  Files         392      393    +1     
  Lines        8259     8273   +14     
=======================================
+ Hits         8019     8033   +14     
  Misses        240      240           

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

@simi
Copy link
Contributor

simi commented Apr 24, 2024

Is there any reason to use jsonb for list value instead of native PostgreSQL array type?

@segiddins segiddins force-pushed the segiddins/store-api-key-scopes-in-jsonb-column branch from 68e6256 to 41bf262 Compare April 24, 2024 21:11
@segiddins
Copy link
Contributor Author

@simi changed to array of string

@segiddins segiddins force-pushed the segiddins/store-api-key-scopes-in-jsonb-column branch from 41bf262 to 86bcf90 Compare April 24, 2024 21:18
@segiddins segiddins marked this pull request as ready for review April 24, 2024 21:18
@segiddins segiddins force-pushed the segiddins/store-api-key-scopes-in-jsonb-column branch from 86bcf90 to 48d8103 Compare April 25, 2024 20:57
@segiddins segiddins force-pushed the segiddins/store-api-key-scopes-in-jsonb-column branch from 48d8103 to a24f43e Compare April 25, 2024 21:21
@segiddins segiddins changed the title Store api key scopes in jsonb column Store api key scopes in array column Apr 26, 2024
@segiddins segiddins merged commit 0f18bd1 into master Apr 26, 2024
@segiddins segiddins deleted the segiddins/store-api-key-scopes-in-jsonb-column branch April 26, 2024 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants