-
Notifications
You must be signed in to change notification settings - Fork 231
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(component): Created calendar input using vcalendar library #218
Conversation
❌ Deploy Preview for activist-org failed.
|
Thank you for the pull request!The activist team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :) If you're not already a member of our public Matrix community, please consider joining! It'd be great to have you! Maintainer checklist
|
Hey @rohram04 👋 Thanks for the PR here :) Not sure if you can see the deploy log, but the error here is the following:
|
That's odd I made sure to install the package. Let me look into it. In the meantime if you want to check out the component you can visit CodeSandbox and run EDIT: Ok there still seems to be an issue. I had previously installed the npm package in the root directory instead of ./frontend. Despite installing the dependency to the frontend directory there still seems to be an issue. I have been a little lazy by not setting up a full docker environment but I will do so to fix this build issue. |
Hey @rohram04! Finally have some time to get to some of these PRs :) How are you doing on this? Do you need some help getting Docker setup for yourself? Overall comments: as I said in the dev sync it's really looking good. I would maybe style it such that the time selection is on the same row rather than one on top of the other (we can do one on top of the other for mobile as we'll have less space). |
Hello, I had some issues using docker. Docker itself is setup but when I run docker-compose up the backend runs fine but the frontend produces the following error log;
It says it's missing some modules. I'm going to look into it myself but just wanted to see if you have seen this before and have any ideas for dealing with it? |
Updated your comment so there’s formatting for the code, @rohram04 :) Will take a look into it! |
@rohram04, glad that Docker is generally working except for this error :) Some suggestions I have:
Let me know if any of these help :) I'm hopeful that just the first one will get us to a working build 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey there @rohram04 👋 Appreciate the work here. I'll bring it in at this point and will work to finish it in the coming weeks :)
Contributor checklist
Description
Created a calendar input form using the vcalendar library as discussed in #169. The API Request needs to be adjusted once a corresponding backend endpoint is created and a calendar to display the events is being made. In it's current form this is not complete or ready to be merged. I am looking for input as I develop the component. The changes have been tested using the following CodeSandbox Environment
Related issue