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.

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

Improve account sharing management #793

Merged
merged 9 commits into from
Jun 4, 2023
Merged

Improve account sharing management #793

merged 9 commits into from
Jun 4, 2023

Conversation

VMelnalksnis
Copy link
Owner

@VMelnalksnis VMelnalksnis commented May 13, 2023

Fixes #226

Changes proposed in this pull request:

  • Fix access control in all queries/operations
  • Add UI for managing access groups, and sharing accounts/transactions with other users

@VMelnalksnis VMelnalksnis added type:enhancement New feature or request area:ui Relating to user interface area:build Relating to deployment/releases area:api labels May 13, 2023
@codecov
Copy link

codecov bot commented May 13, 2023

Codecov Report

Merging #793 (9d918a0) into master (ee99e92) will increase coverage by 0.41%.
The diff coverage is 89.60%.

@@            Coverage Diff             @@
##           master     #793      +/-   ##
==========================================
+ Coverage   75.02%   75.44%   +0.41%     
==========================================
  Files         345      355      +10     
  Lines       10248    10472     +224     
==========================================
+ Hits         7689     7901     +212     
- Misses       2559     2571      +12     
Components Coverage Δ
API 82.26% <93.33%> (+0.23%) ⬆️
UI 59.97% <83.49%> (+1.67%) ⬆️

@VMelnalksnis VMelnalksnis force-pushed the sharing branch 5 times, most recently from fab7c84 to ed9343c Compare May 24, 2023 16:20
@VMelnalksnis VMelnalksnis force-pushed the sharing branch 6 times, most recently from 66fa839 to f6f66cf Compare June 4, 2023 07:09
@VMelnalksnis VMelnalksnis marked this pull request as ready for review June 4, 2023 07:24
@VMelnalksnis VMelnalksnis merged commit 9d918a0 into master Jun 4, 2023
12 checks passed
@VMelnalksnis VMelnalksnis deleted the sharing branch June 4, 2023 07:33
@VMelnalksnis VMelnalksnis temporarily deployed to github-pages June 4, 2023 07:34 — with GitHub Pages Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api area:build Relating to deployment/releases area:ui Relating to user interface type:enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add sharing to UI
1 participant