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

Feat/register page #3

Merged
merged 14 commits into from
Jan 27, 2024
Merged

Feat/register page #3

merged 14 commits into from
Jan 27, 2024

Conversation

kikinnicha
Copy link
Contributor

Change made

  • Bug fixes
  • New features
  • Breaking changes

Describe what you have done

New Features

Fix

Others

@shalluv
Copy link
Member

shalluv commented Jan 10, 2024

For images, according to the Astro docs on where to keep them

We recommend that local images are kept in src/ when possible so that Astro can transform, optimize and bundle them. Files in the /public directory are always served or copied into the build folder as-is, with no processing.

I also recommend following this approach for better performance.

@shalluv
Copy link
Member

shalluv commented Jan 10, 2024

Also, please open a new pull request that only contains assets (no code). This way, I can merge it into the dev branch without having to worry about code review.

Copy link
Member

@shalluv shalluv left a comment

Choose a reason for hiding this comment

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

Quick review.

src/styles/global.css Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
@shalluv
Copy link
Member

shalluv commented Jan 16, 2024

Since we already have src/assets, I suggest we could remove public/{images}.

@shalluv shalluv merged commit 62eb485 into dev Jan 27, 2024
1 check passed
@shalluv shalluv deleted the feat/register-page branch January 27, 2024 17:25
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