Skip to content

Add new fields to CMIS#4239

Merged
zipy124 merged 3 commits intomasterfrom
zipy124-patch-1
Mar 2, 2024
Merged

Add new fields to CMIS#4239
zipy124 merged 3 commits intomasterfrom
zipy124-patch-1

Conversation

@zipy124
Copy link
Copy Markdown
Member

@zipy124 zipy124 commented Mar 2, 2024

New fields added to CMIS

What does this PR do?

For example: Changes the way we retrieve bookings

✅ Pull Request checklist

  • Is this code complete?
  • Are tests done/modified?
  • Are docs written for this PR? (if applicable)

🚨 Is this a breaking change for API clients?

Yes/No

🚀 Deploy notes

For example: Need to run migrations as part of deployment

Anything else

New fields added to CMIS
emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True, blank=True)
field1 = models.CharField(max_length=55, null=True)
field2 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

ema = models.CharField(max_length=1, null=True)
emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True, blank=True)
field1 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True, blank=True)
field1 = models.CharField(max_length=55, null=True)
field2 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

ema = models.CharField(max_length=1, null=True)
emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True, blank=True)
field1 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True)
field1 = models.CharField(max_length=55, null=True)
field2 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

ema = models.CharField(max_length=1, null=True)
emaid = models.TextField(max_length=12, null=True)
dob = models.DateField(null=True)
field1 = models.CharField(max_length=55, null=True)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True not recommended to be used in CharField

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.48%. Comparing base (3139129) to head (cb69674).

❗ Current head cb69674 differs from pull request most recent head 08afb01. Consider uploading reports for the commit 08afb01 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4239      +/-   ##
==========================================
+ Coverage   80.46%   80.48%   +0.02%     
==========================================
  Files         151      151              
  Lines        8615     8620       +5     
==========================================
+ Hits         6932     6938       +6     
+ Misses       1683     1682       -1     

see 2 files with indirect coverage changes

@zipy124 zipy124 merged commit 76dc9e9 into master Mar 2, 2024
@zipy124 zipy124 deleted the zipy124-patch-1 branch March 2, 2024 18:25
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.

1 participant