Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: usermanager #2139

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
Merged
merged 17 commits into from
May 10, 2024
Merged

feat: usermanager #2139

merged 17 commits into from
May 10, 2024

Conversation

@dni
Copy link
Member

@dni dni commented Nov 30, 2023 •
edited
Loading

add usermanager to admin_ui. show user accounts and wallets with transaction count and balance

screenshot-1702563067

  • adds manage user section
  • move topup to users
  • enable / disable admins

dni added a commit that referenced this pull request Nov 30, 2023
@dni
feat: add group_by to fetch_page
3aac26e 
feature is required by #2139
@dni dni mentioned this pull request Nov 30, 2023
Merged
@dni dni changed the base branch from dev to feat-groupby-fetchpage November 30, 2023 22:39
@dni dni force-pushed the feat-usermanager branch from f57836a to 999745d Compare November 30, 2023 22:40
@dni dni mentioned this pull request Dec 11, 2023
Closed
@prusnak prusnak linked an issue Dec 11, 2023 that may be closed by this pull request
[BUG] Reserve Percent & LNbits Balance aren't correct after account deletion #2164
Closed
dni added a commit that referenced this pull request Dec 12, 2023
@dni
feat: add group_by to fetch_page
18d72f6 
feature is required by #2139
@dni dni force-pushed the feat-usermanager branch from e98139a to 14a7b08 Compare December 12, 2023 14:00
dni added a commit that referenced this pull request Dec 12, 2023
@dni
feat: add group_by to fetch_page
4bd81b2 
feature is required by #2139
@dni dni force-pushed the feat-usermanager branch 2 times, most recently from 4948820 to 2c4936b Compare December 14, 2023 10:35
dni added a commit that referenced this pull request Dec 14, 2023
@dni
feat: add group_by to fetch_page
7633e94 
feature is required by #2139
@dni dni added this to the 0.12.0 milestone Dec 14, 2023
@dni dni changed the base branch from feat-groupby-fetchpage to dev December 14, 2023 14:13
@dni dni force-pushed the feat-usermanager branch from 2c4936b to 7031cdd Compare December 14, 2023 14:13
dni added a commit that referenced this pull request Dec 14, 2023
@dni
feat: add group_by to fetch_page
9731964 
feature is required by #2139
dni added a commit that referenced this pull request Dec 14, 2023
@dni
feat: add group_by to fetch_page
16954ab 
feature is required by #2139
@dni dni force-pushed the feat-usermanager branch from 7031cdd to 6e981d9 Compare December 20, 2023 08:52
dni added a commit that referenced this pull request Dec 20, 2023
@dni
feat: add group_by to fetch_page
049e374 
feature is required by #2139
dni added a commit that referenced this pull request Jan 22, 2024
@dni
feat: add group_by to fetch_page
73853bb 
feature is required by #2139
@prusnak prusnak modified the milestones: 0.12.0, 0.12.1 Jan 22, 2024
dni added a commit that referenced this pull request Jan 24, 2024
@dni
feat: add group_by to fetch_page
a1685d4 
feature is required by #2139
@dni dni force-pushed the feat-usermanager branch from 6e981d9 to 0cc5749 Compare January 24, 2024 08:10
@dni dni mentioned this pull request Jan 25, 2024
Closed
@codecov
Copy link

codecov bot commented Jan 25, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 44.26230% with 68 lines in your changes are missing coverage. Please review.

Project coverage is 61.65%. Comparing base (eae5002) to head (ed18108).

Files Patch % Lines
lnbits/core/views/user_api.py 29.41% 60 Missing ⚠️
lnbits/core/crud.py 44.44% 5 Missing ⚠️
lnbits/core/views/generic.py 50.00% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##              dev    #2139      +/-   ##
==========================================
- Coverage   61.84%   61.65%   -0.19%     
==========================================
  Files          67       68       +1     
  Lines        9611     9720     +109     
==========================================
+ Hits         5944     5993      +49     
- Misses       3667     3727      +60

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

@dni dni force-pushed the feat-usermanager branch from 1a055b6 to 02dc54c Compare January 26, 2024 09:29
@dni dni force-pushed the feat-usermanager branch from 02dc54c to 6bf6837 Compare February 7, 2024 07:56
@arcbtc
Copy link
Member

arcbtc commented Feb 9, 2024 •
edited
Loading

A user could spam and make 1000s of wallets, so a delete all button for wallets would be good.

@motorina0 motorina0 removed this from the 0.12.2 milestone Feb 13, 2024
motorina0
motorina0 reviewed May 10, 2024
View reviewed changes
motorina0
motorina0 reviewed May 10, 2024
View reviewed changes
@motorina0 motorina0 mentioned this pull request May 10, 2024
Merged
@dni dni force-pushed the feat-usermanager branch 4 times, most recently from d91979c to 506d121 Compare May 10, 2024 09:32
@dni dni requested a review from arcbtc May 10, 2024 09:46
@dni dni force-pushed the feat-usermanager branch from d89a232 to ed18108 Compare May 10, 2024 09:57
@dni dni merged commit 9ca14f2 into dev May 10, 2024
@dni dni deleted the feat-usermanager branch May 10, 2024 10:06
@motorina0 motorina0 mentioned this pull request Jul 26, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@motorina0 motorina0 motorina0 approved these changes

@arcbtc arcbtc Awaiting requested review from arcbtc

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Reserve Percent & LNbits Balance aren't correct after account deletion

4 participants

@dni @arcbtc @motorina0 @prusnak