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

Major overhaul of Urchin #97

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

AlDu2407
Copy link
Contributor

@AlDu2407 AlDu2407 commented Nov 3, 2024

  • Added TailwindCSS to Urchin.
  • Updated Makefile to download TailwindCSS (currently only Linux is supported)
  • Refactored templates to share similar logic between templates.
  • Redesigned UI for Urchin.
  • Added layout page shared between all sub pages.
  • Added dark mode support.
  • Added new renderHtml method to reduce duplicate code in go.
  • Added self-designed minimalistic 'sea urchin' favicon. :D
  • Added missing sites about and services.
  • Added Not Found handling if a resource is requested that is not present, displaying an error message while keeping the layout page.
  • Restructured static folder to include stylesheets, scripts and assets.

- Added TailwindCSS to Urchin.
- Updated Makefile to download TailwindCSS (currently only Linux is supported)
- Refactored templates to share similar logic between templates.
- Redesigned UI for Urchin.
- Added layout page shared between all sub pages.
- Added dark mode support.
- Added new `renderHtml` method to reduce duplicate code in go.
- Added self-designed minimalistic 'sea urchin' favicon. :D
- Added missing sites `about` and `services`.
- Added `Not Found` handling if a resource is requested that is not present, displaying an error message while keeping the layout page.
- Restructured `static` folder to include stylesheets, scripts and assets.
- Updated Makefile to use `wget` instead of `cURL` to download TailwindCSS
CLI.
- Removed `cURL` from Dockerfile installation.
@matheusgomes28
Copy link
Owner

Hey @AlDu2407 , I wonder if you ever got a chance to add wget to the image?

Btw I think it has curl, so you could work around downloading the stuff you need on CI:

cp -r migrations tests/helpers/
if [ ! -f tailwindcss ]; then \
	wget -q https://github.com/tailwindlabs/tailwindcss/releases/latest/download/tailwindcss-linux-x64 \
		&& chmod +x tailwindcss-linux-x64 \
		&& mv tailwindcss-linux-x64 tailwindcss; \
fi
/bin/sh: 2: wget: not found
make: *** [Makefile:3[7](https://github.com/matheusgomes28/urchin/actions/runs/11663344344/job/32471628007?pr=97#step:4:8): install-tailwindcss] Error 127

@AlDu2407
Copy link
Contributor Author

Hi @matheusgomes28 ,

is there a way for you to remove the line in the Dockerfile, that removes wget from the build image? I think afterwards the pipeline will succeed. And afterwards you should be able to review the entire code.

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