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

Create project create form #65

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

Conversation

kiprotichdominic
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Mar 20, 2023

Visit the preview URL for this PR (updated for commit 831ef18):

https://moringa-library--pr65-create-project-creat-n2dsw0uc.web.app

(expires Mon, 27 Mar 2023 12:47:40 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 28c4ac8715d663d94559951bf1a9d3f1dcb7979f

Copy link
Contributor

@Albert-Byrone Albert-Byrone left a comment

Choose a reason for hiding this comment

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

Kindly note that one check has failed

Copy link
Contributor

@otsembo otsembo left a comment

Choose a reason for hiding this comment

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

Good Job on the PR @kiprotichdominic
Check on the few comments and make necessary fixes.

api/app/controllers/project_controller.rb Outdated Show resolved Hide resolved
api/db/moringa-lib-test.sqlite3 Outdated Show resolved Hide resolved
api/config/environment.rb Outdated Show resolved Hide resolved
Copy link
Contributor

@otsembo otsembo left a comment

Choose a reason for hiding this comment

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

@kiprotichdominic
Copy link
Contributor Author

Good Job @kiprotichdominic

Thanks @otsembo

Description
</label>
<ReactQuill
className="bg-white border border-solid border-gray-300 rounded-lg p-4 w-full"
Copy link
Contributor

Choose a reason for hiding this comment

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

Kindly remove the padding here p-4

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.

3 participants