Skip to content

Conversation

@jeremytchang
Copy link
Collaborator

@jeremytchang jeremytchang commented Nov 1, 2021

  • Add JOIN Yt videos to hackathon resources
  • Updated resource tags and type to use enums for better maintenance
  • Updated resource buttonitem rendering to be based off tags enum

- Updated resource tags and type to use enums for better maintenance
- Updated resource buttonitem rendering to be based off tags enum
@jeremytchang jeremytchang requested a review from jkaster November 1, 2021 22:01
@google-cla google-cla bot added the cla: yes label Nov 1, 2021
@jeremytchang jeremytchang changed the title (fix) Add hackathon resources and refactor to use enums fix: Add hackathon resources and refactor to use enums Nov 1, 2021
Copy link

@jkaster jkaster left a comment

Choose a reason for hiding this comment

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

Nice improvement

}

// Enum serves as display strings too
export enum ResourceTag {
Copy link

Choose a reason for hiding this comment

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

Handier

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

*/

// Enum serves as display strings too
enum ResourceType {
Copy link

Choose a reason for hiding this comment

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

Handy

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Comment on lines +50 to +51
type: ResourceType
tag: ResourceTag
Copy link

Choose a reason for hiding this comment

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

Handiest!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jeremytchang jeremytchang merged commit 59faaab into main Nov 2, 2021
@jeremytchang jeremytchang deleted the add_hackathon_resources branch November 2, 2021 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants