Skip to content

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

Merged
merged 3 commits into from
Apr 18, 2025
Merged

Update beam calls schedule #16

merged 3 commits into from
Apr 18, 2025

Conversation

shariqnaiyer
Copy link
Contributor

#15

Copy link
Collaborator

@unnawut unnawut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all just nitpicks!

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",
Copy link
Collaborator

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

},
{
id: "call-7",
title: "Beam Call #7: PQ Sub-Spec",
title: "Beam Call #7: Rainbow Staking",
status: "unscheduled",
Copy link
Collaborator

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

thumbnail: null,
youtubeUrl: "#",
youtubeUrl: "#"
Copy link
Collaborator

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

thumbnail: null,
youtubeUrl: "#",
youtubeUrl: "#"
Copy link
Collaborator

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

@@ -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: "#"
Copy link
Collaborator

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

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: "#"
Copy link
Collaborator

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

},
]
]
Copy link
Collaborator

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

{
id: "call-13",
title: "Beam Call #13: Beam Spec (Part 2)",
status: "unscheduled",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upcoming

thumbnail: null,
youtubeUrl: "#",
youtubeUrl: "#"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trailing comma

@shariqnaiyer
Copy link
Contributor Author

Thank you for the comments. I believe my latest commit fixes the listed inconsitencies.

@shariqnaiyer shariqnaiyer requested a review from unnawut April 18, 2025 18:19
Copy link
Collaborator

@unnawut unnawut left a 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!

@unnawut unnawut merged commit 76a4a04 into ReamLabs:master Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants