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

[C-API] Add relaxed schema configuration option #7864

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

rorbech
Copy link
Contributor

@rorbech rorbech commented Jul 5, 2024

What, How & Why?

Expose new configuration option to C-API

RLM_API void realm_config_set_flexible_schema(realm_config_t* realm_config,
                                             bool flexible_schema) noexcept
{
    realm_config->flexible_schema = flexible_schema;
}

☑️ ToDos

  • 📝 Changelog update
  • 🚦 Tests (or not relevant) Considered to be tested as part of the overall feature
  • C-API, if public C++ API changed
  • bindgen/spec.yml, if public C++ API changed

@rorbech rorbech requested a review from jedelbo July 5, 2024 11:03
@cla-bot cla-bot bot added the cla: yes label Jul 5, 2024
@rorbech rorbech changed the title Expose relaxed schema configuration to C-API configuration [C-API] Add relaxed schema configuration option Jul 5, 2024
@rorbech rorbech added no-changelog no-jira-ticket Skip checking the PR title for Jira reference labels Jul 5, 2024
@rorbech rorbech force-pushed the cr/flexible-schema-capi-configuration branch from 73198bc to f49f8ee Compare July 8, 2024 08:46
@jedelbo jedelbo merged commit 2ae9b7d into je/flexible-schema Jul 8, 2024
10 of 35 checks passed
@jedelbo jedelbo deleted the cr/flexible-schema-capi-configuration branch July 8, 2024 12:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes no-changelog no-jira-ticket Skip checking the PR title for Jira reference
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants