-
Notifications
You must be signed in to change notification settings - Fork 10
AIP #61 v5: Channel and Campaign #382
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
Closed
21 tasks done
Comments
This was referenced Mar 9, 2021
elpiel
added a commit
that referenced
this issue
Mar 18, 2021
AdEx v5: Issue #382 Channel and Campaign
elpiel
added a commit
that referenced
this issue
Apr 9, 2021
4 tasks
elpiel
added a commit
that referenced
this issue
May 25, 2021
Issue #382 Channel and Campaign Database & Persistance
This was referenced Jun 18, 2021
3 tasks
Merged
This was referenced Jan 18, 2022
Merged
3 tasks
This was
linked to
pull requests
Apr 13, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
AIP AmbireTech/adex-validator-stack-rust#61
Part of #377
Channel
As AIP#61 states the Channel is defined by
(leader, follower, guardian, tokenAddr, nonce)
channel_id
in thecampaigns
table)Campaign
Campaign { id: CampaignId, channel: Channel, creator, validators, title.... rest of the ChannelSpec fields }
Note: The fields were aligned with Ivo before making the changes for the Campaign.
CampaignId
should be a randomly assigned value bySentry
, we've chosen theUuid
as a random generator in Rust and uses a representation of the 16 bytes as0x
prefixed hex stringNo migration
Apart from that, we can remove obsolete fields in the
ChannelSpec
(v4) as the v5 will not have migration from v4 as it will work in a different way than v4.Tasks:
Sentry
Sentry routes
Clean up V4 Channel and other routes & functions Issue 382 Remove V4 Channel and fns cleanup #435
POST (create)
Campaign
AIP #61 v5: Sentry POST route for inserting/modifying campaign #409 (PR V5: Authenticated POST route to modify/submit campaigns #408 )spendable
by calling theadapter.get_deposit
and updating it in the DBwhoami
validator in the channel & campaign validators, etc.)PUT (update)
Campaign
AIP #61 v5: Sentry POST route for inserting/modifying campaign #409 (PR V5: Authenticated POST route to modify/submit campaigns #408 )spendable
by calling theadapter.get_deposit
and updating it in the DBCampaignId
&Campaign.channel
is mutableGET (list) Campaigns AIP #61 v5: GET (list) Campaigns #429 (PR GET /v5/campaign/list #430 ) @simzzz
See how we've done this for the V4
GET /channel/list
validator
- Address (not mandatory) ORleader
- Address (not mandatory)adview-manager
updatePOST ../events
route Adview Manager V5 changes #466 @simzzzGET (list) Channel - V5 Tests Tests for GET /v5/channel/list and multichain support #479
(In sentry) Use a separate
channels
table since we will need it for look-up when for example we want to get the lastApproveState
we need to know now only theChannel.id
(/channel/:id/...
from the route) but also theChannel.leader
to select the last approvedNewState
of the Leader (first to fetch theApproveState
and then to get the correspondingNewState
)POST /v5/campaign/events - insert new Event(s)
Analytics V5
DateHour
(& FromSql & ToSql impls)sentry::analytics::record()
&sentry::db::update_analytics
with more diverse Event & params of Events (like setting hostname, referrer, etc.)The text was updated successfully, but these errors were encountered: