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

Rename to completed_on and requested_on in the CustomFormModel #11592

Merged
merged 4 commits into from
May 26, 2020

Conversation

rakshith91
Copy link
Contributor

@rakshith91 rakshith91 commented May 21, 2020

Part of #11283

@rakshith91 rakshith91 marked this pull request as ready for review May 21, 2020 22:22
@@ -65,8 +65,8 @@ async def manage_custom_models(self):
custom_model = await form_training_client.get_custom_model(model_id=first_model.model_id)
print("Model ID: {}".format(custom_model.model_id))
print("Status: {}".format(custom_model.status))
print("Created on: {}".format(custom_model.created_on))
print("Last modified: {}".format(custom_model.last_modified))
print("Created on: {}".format(custom_model.requested_on))
Copy link
Contributor

Choose a reason for hiding this comment

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

Should switch the text in these print statements too

@@ -16,7 +16,9 @@
- `USReceiptType` is renamed to `ReceiptType`
- `use_training_labels` is now a required positional param in the `begin_training` APIs.
- `stream` and `url` parameters found on methods for `FormRecognizerClient` have been renamed to `form` and `form_url`, respectively.
For recognize receipt methods, parameters have been renamed to `receipt` and `receipt_url`.
- For recognize receipt methods, parameters have been renamed to `receipt` and `receipt_url`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should also update the readme, i.e. here (i think there are two code snippets that need updating): https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/formrecognizer/azure-ai-formrecognizer/README.md#train-a-model

@rakshith91 rakshith91 force-pushed the custom_form_created branch from 59feb11 to d6a9ff8 Compare May 21, 2020 23:45
For recognize receipt methods, parameters have been renamed to `receipt` and `receipt_url`.
- For recognize receipt methods, parameters have been renamed to `receipt` and `receipt_url`.
- `created_on` and `last_modified` are renamed to `requested_on` and `completed_on` in the
`CustomForm` models.
Copy link
Member

Choose a reason for hiding this comment

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

I would spell out the models that changed since there were only two

Co-authored-by: Krista Pratico <[email protected]>
@rakshith91 rakshith91 merged commit b94b42f into Azure:master May 26, 2020
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request May 27, 2020
…into models_to_submodels

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  [text analytics] merging feature branch into master (Azure#11632)
  regen swagger and make list_indexes pageable (Azure#11635)
  Sync eng/common directory with azure-sdk-tools repository (Azure#11566)
  Rename to completed_on and requested_on in the CustomFormModel (Azure#11592)
  [formrecognizer] model_id param validation (Azure#11569)
  [Batch] Fix issue in latest REST API (Azure#11604)
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