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.

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

User registration step in annotation tools #2659

Merged
merged 11 commits into from
Oct 21, 2024
Merged

User registration step in annotation tools #2659

merged 11 commits into from
Oct 21, 2024

Conversation

portuu3
Copy link
Collaborator

@portuu3 portuu3 commented Oct 21, 2024

Description

If the annotation tool requires a user registration before annotate data, users should see an extra step in HUMAN App that drives them towards registration process.

Summary of changes

  • Add registration step to HUMAN App frontend.
  • Add registartionNeeded and registrationInstructions to leader entity in subgraph.
  • Update SDK based on subgraph changes.
  • Post a request from HUMAN App to reputation oracle to save the registration completion.

How test the changes

  1. Set registration needed for some exchange oracle
  2. Go to see the jobs of oracle. You should be required to go through the registration step.

portuu3 and others added 8 commits October 17, 2024 15:09
@portuu3
[HUMAN App client] Add registration step (#2518)
55b1dff 
* Add logic to register users in oracles

* Fix failing tests
---------

Co-authored-by: portuu3
@portuu3
Revert "Bump solidity-coverage from 0.8.12 to 0.8.13 (#2605)" (#2655)
b93051c 
This reverts commit f531a68.
@portuu3
Merge branch 'develop' into feature/registration
90822b7
@portuu3
Merge branch 'develop' into feature/registration
461add7
@portuu3 @flopez7
[Subgraph][SDK] Add registration data to leaders entity (#2451)
724cb2e 
* Add registration data to leaders entity

* Add registration info to getReputationNetworkOperators

---------

Co-authored-by: Francisco López <[email protected]>
@flopez7 @portuu3
[Human App] Add registration info to oracles discovery response (#2452)
95ed623 
* Add registration data to leaders entity

* Add registration info to getReputationNetworkOperators

* Add wether registration is required to oracles discovery response
@portuu3
update docs
eafa29d
@portuu3
fix human app frontend and rep oracle user service for registration
b9047f7
@portuu3 portuu3 self-assigned this Oct 21, 2024
@vercel Vercel
Copy link

vercel bot commented Oct 21, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
human-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 21, 2024 11:40am
human-dashboard-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 21, 2024 11:40am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Visit Preview Oct 21, 2024 11:40am
faucet-server ⬜️ Ignored (Inspect) Visit Preview Oct 21, 2024 11:40am

@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend October 21, 2024 09:48 Inactive
@Dzeranov Dzeranov self-requested a review October 21, 2024 11:01
@portuu3 portuu3 merged commit 61e6631 into develop Oct 21, 2024
25 checks passed
@portuu3 portuu3 deleted the feature/registration branch October 21, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dzeranov Dzeranov Dzeranov approved these changes

Assignees

@portuu3 portuu3

Labels
None yet
Projects
Core-tech - 2024
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@portuu3 @Dzeranov @flopez7