-
Notifications
You must be signed in to change notification settings - Fork 7
Update beam calls schedule #16
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all just nitpicks!
data/beam-calls.tsx
Outdated
summary: | ||
"Exploration of the Attester-Proposer Separation (APS) concept, its benefits, and potential applications for the Beam Chain.", | ||
thumbnail: null, | ||
youtubeUrl: "#", | ||
}, | ||
{ | ||
id: "call-6", | ||
title: "Beam Call #6: Rainbow Staking", | ||
title: "Beam Call #6: 3SF (Finality)", | ||
status: "unscheduled", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change to upcoming if date is set
data/beam-calls.tsx
Outdated
}, | ||
{ | ||
id: "call-7", | ||
title: "Beam Call #7: PQ Sub-Spec", | ||
title: "Beam Call #7: Rainbow Staking", | ||
status: "unscheduled", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change to upcoming if date is set
data/beam-calls.tsx
Outdated
thumbnail: null, | ||
youtubeUrl: "#", | ||
youtubeUrl: "#" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: please add back the trailing comma for consistency
data/beam-calls.tsx
Outdated
thumbnail: null, | ||
youtubeUrl: "#", | ||
youtubeUrl: "#" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: please add back the trailing comma for consistency
data/beam-calls.tsx
Outdated
@@ -208,26 +225,36 @@ export const beamCallsData: BeamCall[] = [ | |||
summary: | |||
"Exploring the Attester-Proposer Separation (APS) sub-specification, its benefits, and potential applications in the Beam Chain.", | |||
thumbnail: null, | |||
youtubeUrl: "#", | |||
youtubeUrl: "#" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: please add back the trailing comma for consistency
data/beam-calls.tsx
Outdated
summary: | ||
"Second part of the Beam specification discussion, focusing on the protocol's advanced features, security considerations, and future development plans.", | ||
thumbnail: null, | ||
youtubeUrl: "#", | ||
youtubeUrl: "#" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: please add back the trailing comma for consistency
data/beam-calls.tsx
Outdated
}, | ||
] | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add back the newline
data/beam-calls.tsx
Outdated
{ | ||
id: "call-13", | ||
title: "Beam Call #13: Beam Spec (Part 2)", | ||
status: "unscheduled", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upcoming
data/beam-calls.tsx
Outdated
thumbnail: null, | ||
youtubeUrl: "#", | ||
youtubeUrl: "#" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trailing comma
Thank you for the comments. I believe my latest commit fixes the listed inconsitencies. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much cleaner. Many thanks!
#15