Skip to content

171784102 Add Create accommodation Feature #15

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Francois-MUGOROZI
Copy link
Collaborator

What this PR does?

  • Delivers create accommodation facility resources into the project.

Description of the task to be completed?

  • add UI for user input form
  • integrate with backend API
  • add tests
  • upload image to Cloudinary

How should this be tested?

Clone the repo and install all dependencies.
login to get access and then go to /create-accommodation in your browser

What are the relevant pivotal tracker stories?

#171784102

Screenshots?

Web view:
Responsiveness

@Francois-MUGOROZI Francois-MUGOROZI added the WIP Work in progress label Jun 25, 2020
@Francois-MUGOROZI Francois-MUGOROZI force-pushed the ft-create-accommodation-171784102 branch 2 times, most recently from 36cf2ce to 012293e Compare June 25, 2020 19:31
@Francois-MUGOROZI Francois-MUGOROZI added Ready For Review Requires code review from Dev team and removed WIP Work in progress labels Jun 25, 2020
@Francois-MUGOROZI Francois-MUGOROZI force-pushed the ft-create-accommodation-171784102 branch 4 times, most recently from 74fe3f2 to 5d61f3a Compare June 29, 2020 06:00
padding: 0;
box-shadow: $rightShadow;

// @media (max-height:800px) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the commented code could be deleted.

- add ui
- connect with backend
- add tests
[Finish]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review Requires code review from Dev team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants