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

feat: field convert limit #848

Merged
merged 3 commits into from
Aug 22, 2024
Merged

feat: field convert limit #848

merged 3 commits into from
Aug 22, 2024

Conversation

caoxing9
Copy link
Contributor

No description provided.

@coveralls
Copy link
Collaborator

coveralls commented Aug 20, 2024

Pull Request Test Coverage Report for Build 10509485927

Details

  • 207 of 232 (89.22%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on feat/field-convert-limit at 83.359%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/nestjs-backend/src/features/field/field-calculate/field-view-sync.service.ts 182 207 87.92%
Totals Coverage Status
Change from base Build 10487822919: 83.4%
Covered Lines: 30712
Relevant Lines: 36843

💛 - Coveralls

@caoxing9 caoxing9 merged commit c246feb into develop Aug 22, 2024
7 checks passed
@caoxing9 caoxing9 deleted the feat/field-convert-limit branch August 22, 2024 14:06
AliceLanniste pushed a commit to AliceLanniste/teable that referenced this pull request Sep 1, 2024
* feat: abstract common action for enterprise field convert

* chore: add delete or convert field view condition change e2e

* perf: detach the field relative address service to `FieldViewSyncService`
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