Skip to content

Conversation

ThibautGeriz
Copy link
Contributor

@ThibautGeriz ThibautGeriz commented Mar 17, 2020

We a more complex use case on the photo classification, I made the following changes:

  • add a custom prop EditForm to provide a custom react component
  • add the props in the custom RegionLabel: regions & imageSrc

Questions:

  • are you happy with the name EditForm?
  • I have added in the state, should I leave it in the props?
@ThibautGeriz ThibautGeriz force-pushed the feat/custom-region-form branch from fcedbbe to d20b9a8 Compare March 17, 2020 05:04
@ThibautGeriz ThibautGeriz changed the title Feat/custom region form [WIP] Feat/custom region form Mar 17, 2020
@seveibar
Copy link
Contributor

Hey @ThibautGeriz, I like how in this implementation you allow replacing the "RegionLabel" component. tbh I don't love having EditForm in the state, I might prefer RegionEditLabel. This PR hit at a bad time as I just did extensive refactoring to the components.

I'd like to help refactor this or have someone on the wao.ai team help. I'll respond later with more details.

Thanks so much for the contribution! We'll get this merged :)

@ThibautGeriz ThibautGeriz force-pushed the feat/custom-region-form branch from d3b8f38 to 7ac84b9 Compare March 17, 2020 23:11
@ThibautGeriz
Copy link
Contributor Author

@seveibar I've done the requested updates. Let me know when you have time for a full time code review.

@seveibar
Copy link
Contributor

seveibar commented Apr 3, 2020

This looks good to me.

@seveibar
Copy link
Contributor

seveibar commented Apr 3, 2020

So I'm going to merge this but we'll need to do just a bit of testing, then it'll go into the next release.

@seveibar seveibar merged commit 0d664ce into UniversalDataTool:master Apr 3, 2020
stajahlee pushed a commit to skiplist-eng/react-image-annotate that referenced this pull request Apr 3, 2020
seveibar pushed a commit that referenced this pull request Apr 11, 2020
# 1.0.0 (2020-04-11) ### Bug Fixes * introduce semantic release ([a1b3dd8](a1b3dd8)) ### Features * Feat/custom region form ([#32](#32)) ([0d664ce](0d664ce))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants