Skip to content

converted from .tsx to .js for announcements - #10

Open
jennyha18 wants to merge 22 commits into
mainfrom
jpha
Open

converted from .tsx to .js for announcements#10
jennyha18 wants to merge 22 commits into
mainfrom
jpha

Conversation

@jennyha18

Copy link
Copy Markdown

No description provided.

@darreniyer0601 darreniyer0601 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You'll want to pull from the main branch. It looks like you have .js context files, the current updated version should have .tsx files for context. After pulling from main, delete the .js files in /context and use the .tsx file instead.

The announcements.js is fine for now.
I'd highly recommend using jsx instead of React.createElement in the future, it's far simpler and more intuitive. If you haven't used jsx before, you can take a look at how it's done in this page

You can create a new pull request once you've made these changes, and you should be good after that.

@vercel

vercel Bot commented Aug 25, 2022

Copy link
Copy Markdown

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

Name Status Preview Updated
hw22-frontend ❌ Failed (Inspect) Sep 15, 2022 at 5:02AM (UTC)

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.

3 participants