Skip to content
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

fix/content #33

Merged
merged 4 commits into from
Jun 11, 2024
Merged

fix/content #33

merged 4 commits into from
Jun 11, 2024

Conversation

mateusriff
Copy link
Collaborator

@mateusriff mateusriff commented Jun 10, 2024

Fix and Update Content

Changes

  • Updated results: results tables were updated with LARC 2023 data.
  • Fixed VSSS category subtitle: the subtitle is now translated into English.
  • Added publications: five new publications were added.

How to test

Before testing, you must have Node and the Node Package Manager (NPM) installed on your machine.

Initial Setup

  1. Clone this repository by doing git clone https://github.com/robocin/robocin-website.git;
  2. Install dependencies with npm install --legacy-peer-deps;
  3. Switch to this branch with git checkout fix/content;
  4. Do npm run dev to start the application locally;
  5. Go to http://localhost:3000.

Tests

Just make sure the changes made here fix the issues raised by here 👍

@mateusriff mateusriff requested a review from JoaoGMSF June 10, 2024 22:35
@mateusriff mateusriff self-assigned this Jun 10, 2024
Copy link

vercel bot commented Jun 10, 2024

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

Name Status Preview Comments Updated (UTC)
robocin ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 11, 2024 11:23pm

@@ -39,7 +39,7 @@ const en = {
An SSL match unfolds between two teams, each comprised of six
robots. These robots are constrained by specific dimensions;
they must fit within a 180 mm diameter circle and should not
exceed 15 cm in size. Notably, our vision system, which processes
exceed 150 mm in size. Notably, our vision system, which processes
Copy link

Choose a reason for hiding this comment

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

Aqui acho que ficaria melhor trocar "size" por "height".

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Alteração feita ✅

@mateusriff mateusriff merged commit c0dc57f into main Jun 11, 2024
3 checks passed
@mateusriff mateusriff removed the request for review from JoaoGMSF June 11, 2024 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants