grassrootseconomics/sarafu-vise
8
0
Fork 0
Code Issues 15 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

self pin reset #49

Merged
Alfred-mk merged 32 commits from self-pin-reset-node into master 2025-04-11 09:26:11 +02:00
Conversation 1 Commits 32 Files Changed 46
+170 -526
Alfred-mk commented 2025-04-02 17:30:49 +02:00
Member
Copy Link
Copy Source

This resolves issue #48

This resolves issue https://git.grassecon.net/grassrootseconomics/sarafu-vise/issues/48
Alfred-mk added 2 commits 2025-04-02 17:30:50 +02:00
added a DATA_SELF_PIN_RESET to track when a user's PIN is reset by an admin c5673b339b
use the DATA_SELF_PIN_RESET to set the flag_account_pin_reset 2c671bc0d4
Alfred-mk marked the pull request as work in progress 2025-04-02 17:30:54 +02:00
Alfred-mk added 1 commit 2025-04-08 15:23:53 +02:00
updated the ResetOthersPin and ValidateBlockedNumber for the updated reset other's PIN functionality f96f9c11e6
Alfred-mk added 4 commits 2025-04-09 14:10:29 +02:00
move to the authorize_reset_others_pin if the given number is valid and registered d094af9c51
request the admin to authorize the PIN reset 8a6659a98b
move back to the previous node if the admin user is not authorized 514f6ae05b
set the DATA_SELF_PIN_RESET as 0 and reset the flag_account_pin_reset 33c376c971
Alfred-mk added 1 commit 2025-04-09 14:11:56 +02:00
use the self_reset_pin node to reset PINs for accounts with a flag_account_pin_reset 477f3a307a
Alfred-mk added 1 commit 2025-04-09 14:12:58 +02:00
use the confirm_self_pin_reset to confirm the new PIN 71ef950fff
Alfred-mk added 1 commit 2025-04-09 14:19:57 +02:00
move to the top node when one selects back aec96ce9ba
Alfred-mk added 2 commits 2025-04-09 15:55:57 +02:00
allow the user to quit and move to invalid_pin node if the given input is invalid f8c258a3b4
use a single pin_mismatch node for a confirmed PIN that does not match the original PIN 067c496244
Alfred-mk added 1 commit 2025-04-09 15:57:38 +02:00
have the catch for create_pin above to catch accounts that do not have a PIN before the status on chain is checked 29d94bb2e5
Alfred-mk added 2 commits 2025-04-09 15:59:59 +02:00
updated the description of the flag_incorrect_pin 0c1d9ab582
updated the flags set and reset in SaveTemporaryPin and VerifyCreatePin 1f6bf2bbed
Alfred-mk added 2 commits 2025-04-09 16:39:52 +02:00
add a space 26353bdf6e
remove code related to admins resetting a user's actual PIN 834f2ce629
Alfred-mk added 2 commits 2025-04-09 16:53:11 +02:00
updated the menu flow for the admin reset others PIN in test 35a2732fe2
updated the expected content for the alias in view profile a27d44e561
Alfred-mk added 1 commit 2025-04-09 16:57:58 +02:00
use the correct flag_invalid_pin in TestSaveTemporaryPin 1076a9578e
Alfred-mk added 2 commits 2025-04-09 17:06:34 +02:00
update the reset flags in TestConfirmPinChange 4ef8c47f8b
update the reset flags in TestCheckBlockedStatus b89abf3487
Alfred-mk added 1 commit 2025-04-09 17:10:57 +02:00
update the order for the INCMP statement 69ae494b2c
Alfred-mk added 1 commit 2025-04-09 17:14:17 +02:00
cleanup: remove extra space 6070792fe4
Alfred-mk added 1 commit 2025-04-09 17:18:04 +02:00
cleanup: remove extra space in all .vis files 0337c66f96
Alfred-mk added 2 commits 2025-04-09 18:23:53 +02:00
use the same check for flag_invalid_pin to move to the invalid_pin node d680387ef1
use a single confirm_pin_change node aec1f4c4c1
Alfred-mk added 1 commit 2025-04-09 18:29:52 +02:00
remove unused function reset_valid_pin 7d9c3b66a9
Alfred-mk added 1 commit 2025-04-09 22:40:30 +02:00
remove the verify_new_pin functionality that managed the unused flag_valid_pin 57a1645c03
Alfred-mk added 1 commit 2025-04-09 22:43:48 +02:00
Merge branch 'master' into self-pin-reset-node c9170ca45a
Alfred-mk marked the pull request as ready for review 2025-04-09 22:43:57 +02:00
carlos reviewed 2025-04-10 09:37:43 +02:00
store/db/db.go Outdated
@@ -65,6 +65,8 @@ const (
DATA_ACCOUNT_ALIAS
//currently suggested alias by the api awaiting user's confirmation as accepted account alias
DATA_SUGGESTED_ALIAS
//currently suggested alias by the api awaiting user's confirmation as accepted account alias
carlos commented 2025-04-10 09:37:43 +02:00
Member
Copy Link
Copy Source

This comment belongs to the DATA_SUGGESTED_ALIAS

This comment belongs to the DATA_SUGGESTED_ALIAS
Alfred-mk marked this conversation as resolved
Alfred-mk added 1 commit 2025-04-10 12:55:21 +02:00
updated the description of DATA_SELF_PIN_RESET 56cda36aa7
Alfred-mk added 1 commit 2025-04-11 09:25:56 +02:00
Merge branch 'master' into self-pin-reset-node 4ffb32f14a
Alfred-mk merged commit fa9f1a72ef into master 2025-04-11 09:26:11 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
carlos
Labels
Clear labels
ussd/user-reported
Issue reported by a user while interacting with the ussd
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Alfred-mk (Alfred Kamanda) amina carlos kamikazechaser (Mohamed Sohail) lash mango-habanero williamluke willruddick
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: grassrootseconomics/sarafu-vise#49