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

Support for JSON schema #1140

Merged
merged 39 commits into from
Oct 11, 2023
Merged

Support for JSON schema #1140

merged 39 commits into from
Oct 11, 2023

Conversation

pavjacko
Copy link
Member

@pavjacko pavjacko commented Oct 9, 2023

Description

  • Add support for fully typed json schema
  • Unify TS, Docs, Schema into one solution
Oct-06-2023.11-06-15.mp4

Breaking Changes

  • PRs should not introduce breaking changes to existing functionality
  • if breaking change cannot be avoided it has to be introduced in 2 phases (release cycles of 0.x.0)
    • 0.x.0 Add new functionality + add DEPRECATED warning to existing fuctionality
    • 0.[x+1].0 Remove deprecated functionality

I have tested my changes on:

ReNative project directly:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

New project:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

Existing Project created with previous version of renative:

  • ios simulator
  • ios device
  • android simulator
  • android device
  • web browser
  • tvos simulator
  • tvos device
  • androidtv simulator
  • androidtv device
  • androidwear simulator
  • androidwear device
  • tizen simulator
  • tizen device
  • tizenmobile simulator
  • tizenwatch device
  • webos simulator
  • webos device
  • macos
  • windows
  • chromecast device

@pavjacko pavjacko added this to the 1.0 milestone Oct 9, 2023
@pavjacko pavjacko self-assigned this Oct 9, 2023
@pavjacko pavjacko marked this pull request as draft October 9, 2023 01:31
@pavjacko pavjacko changed the title Support for json schema Support for JSON schema Oct 9, 2023
* main:
  Bump electron from 26.2.1 to 26.2.4
  added test for rn.start
  remove unecessary try
  remove cl
  remove cl
  move device logic before bundler, fix sudden bundler exit if failed
  fix: lock gemfile to working activesupport version
@pavjacko pavjacko marked this pull request as ready for review October 9, 2023 10:56
@@ -107,7 +108,7 @@ const runCocoaPods = async (c: Context) => {
return Promise.reject(`Location ${appFolder} does not exists!`);
}
const podsRequired = c.program.updatePods || (await checkIfPodsIsRequired(c));
const permissions = c.buildConfig.permissions?.[c.platform];
const permissions = c.platform === 'ios' ? c.buildConfig.permissions?.[c.platform] : {};
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't we compare it with the constant?

Copy link
Member Author

Choose a reason for hiding this comment

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

@mihaiblaga89 nope. RnvPlatform is TS union type not a string. no need for extra constants.

Screenshot 2023-10-11 at 16 20 01

@pavjacko pavjacko merged commit 3812bb5 into main Oct 11, 2023
3 checks passed
@pavjacko pavjacko deleted the feat/json-schema branch February 14, 2024 19:38
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