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: "docker compose up: issue because of canvas package solved #641

Closed

Conversation

MrUnknown58
Copy link

PR Fixes:

  • 1 docker compose up command was giving an error
    trailing comma
    docker issue

Resolve:
The issue was because of canvas package, it internally used some packages which wasn't correctly installed in docker. I have made changes in the Dockerfile and have added those, now it works prefectly. The minor change in the CourseCard is coz Button and ChevronRight wasn't imported in the file, have commented it for the time being.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@MrUnknown58 MrUnknown58 changed the title fix: docker issue because of canvas package Imp fix: "docker compose up: issue because of canvas package solved May 15, 2024
@MrUnknown58 MrUnknown58 changed the title Imp fix: "docker compose up: issue because of canvas package solved Fix: "docker compose up: issue because of canvas package solved May 16, 2024
@hkirat
Copy link
Contributor

hkirat commented May 21, 2024

we've gotten rid of the canvas library altogether now

@hkirat hkirat closed this May 21, 2024
@MrUnknown58 MrUnknown58 deleted the fix_docker_compose_up_error branch May 22, 2024 06:28
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.

None yet

2 participants