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

Move endpoints related to houses from community tag #232

Open
mslowiak opened this issue Feb 18, 2021 · 6 comments
Open

Move endpoints related to houses from community tag #232

mslowiak opened this issue Feb 18, 2021 · 6 comments
Assignees
Labels
good first issue Good for newcomers up-for-grabs up-for-grabs

Comments

@mslowiak
Copy link
Collaborator

Things to be done:

  • Move GET ​/communities​/{communityId}​/houses from communities tag to houses tag in api.yaml
  • Move POST ​/communities​/{communityId}​/houses from communities tag to houses tag in api.yaml
  • Move DELETE ​/communities/{communityId}/houses/{houseId} from communities tag to houses tag in api.yaml
  • Generate code from new specs
  • Fix test classes
@JanneSickert
Copy link
Contributor

I do this

@deepdmistry
Copy link

Can i take up this task ?

@Murad013
Copy link

Hi @mslowiak, I'm still new to Open Source so I'm confused if this is still open or not. I can see from the tag that it is but it seems like @JanneSickert closed it.

@mslowiak
Copy link
Collaborator Author

Hi @Murad013,
Actually the owner of this project is not responding so I don't know :)
I can assign this to you

@Murad013
Copy link

Thank you!

@mslowiak mslowiak assigned Murad013 and unassigned JanneSickert Sep 25, 2022
Murad013 added a commit to Murad013/MyHome that referenced this issue Sep 26, 2022
Dark-Ultron added a commit to Dark-Ultron/MyHome that referenced this issue Sep 18, 2023
…unity tag and moved associated methods from CommunityController to HouseController
@Dark-Ultron
Copy link

Hi @mslowiak, this issue seemed open, so I tried to fix it without getting assigned. I'm new to Open Source, I'm still learning. Can you please review this PR and merge it if everything is okay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers up-for-grabs up-for-grabs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants