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

Added keyword to string mapping in reports and ui #326

Merged
merged 4 commits into from
Apr 16, 2024

Conversation

rutujaac
Copy link
Collaborator

@rutujaac rutujaac commented Apr 16, 2024

Added the following topic & entity keyword to name mapping in Reports and UI:

"normal-text": "Normal Text",
"medical-advice": "Medical Advice",
"harmful-advice": "Harmful Advice",
"board-meeting": "Board Meeting",
"consulting-agreement": "Consulting Agreement",
"customer-list": "Customer List",
"distribution/partner-agreement": "Distribution/Partner Agreement",
"enterprise-agreement": "Enterprise Agreement",
"executive-severance-agreement": "Executive Severance Agreement",
"financial-report": "Financial Report",
"loan-and-security-agreement ": "Loan And Security Agreement",
"merger-agreement": "Merger Agreement",
nda: "NDA",
"patent-application-fillings": "Patent Application Fillings",
"price-list": "Price List",
"settlement-agreement": "Settlement Agreement",
"sexual-harassment": "Sexual Harassment",
"employee-agreement": "Employee Agreement",
"us-ssn": "US SSN",
"us-passport-number": "US Passport number",
"us-drivers-license": "US Drivers License",
"credit-card-number": "Credit card number",
"us-bank-account-number": "US Bank Account Number",
"iban-code": "IBAN code",
"us-itin": "US ITIN",
"github-token": "Github Token",
"slack-token": "Slack Token",
"aws-access-key": "AWS Access Key",
"aws-secret-key": "AWS Secret Key",
"azure-key-id": "Azure Key ID",
"azure-client-secret": "Azure Client Secret",
"google-api-key": "Google API Key"

NOTE: Ensured backward compatibility - if complete string like "AWS Secret Key" is found instead of keyword "aws-secret-key", original string is rendered.

@rutujaac rutujaac requested a review from shreyas-damle April 16, 2024 11:13
"financial-report": "Financial Report",
"loan-and-security-agreement ": "Loan And Security Agreement",
"merger-agreement": "Merger Agreement",
nda: "NDA",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why this nda is without quotes?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since "nda" is a single (no space) object key, the quotes do not appear on it. We only need quotes around object keys if there are multiple words separated by a space.

@shreyas-damle shreyas-damle merged commit 88835c6 into daxa-ai:main Apr 16, 2024
11 of 15 checks passed
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.

2 participants