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

docs: Add docs (build) #2

Merged
merged 1 commit into from
May 16, 2023
Merged

docs: Add docs (build) #2

merged 1 commit into from
May 16, 2023

Conversation

sandrina-p
Copy link
Collaborator

@sandrina-p sandrina-p commented May 16, 2023

This library was part of @remotecom internal source code.

As we are open-sourcing it, this is the docs (build version).

The library code itself is copied at #1 .

Why not open-sourcing the docs' source?

Although the library code is open-sourced with unit tests, the docs cannot be moved yet. Our docs are still coupled to Remote's internal Design System and integration tests. The effort to decouple it at the moment is too high.

So our strategy is to at each new PR, to update the docs internally and manually copy its build to this repo.

How to review this

  • Check the website where the docs are hosted (Vercel) is working well

@vercel
Copy link

vercel bot commented May 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
json-schema-form ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 16, 2023 4:11pm

@sandrina-p sandrina-p requested a review from brennj May 16, 2023 15:48
@sandrina-p sandrina-p mentioned this pull request May 16, 2023
Copy link
Contributor

@johnstonbl01 johnstonbl01 left a comment

Choose a reason for hiding this comment

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

Docs site looks great to me 👍
Clicked around and played with several pieces. Love that it will show the updates at the bottom of the PR for testing.

@sandrina-p
Copy link
Collaborator Author

Docs site looks great to me 👍 Clicked around and played with several pieces. Love that it will show the updates at the bottom of the PR for testing.

Cool! Which "updates at the bottom" are you talking about?

@remotecom remotecom merged commit c01a5b1 into main May 16, 2023
@johnstonbl01
Copy link
Contributor

Docs site looks great to me 👍 Clicked around and played with several pieces. Love that it will show the updates at the bottom of the PR for testing.

Cool! Which "updates at the bottom" are you talking about?

Screen Shot 2023-05-16 at 1 38 41 PM

sandrina-p added a commit that referenced this pull request May 17, 2023
This commit was amended in the "main" branch to add the co-authors,
which caused the lost of the Merge commit from #2

Co-authored-by: António Capelo <antoniocapelo@users.noreply.github.com>
Co-authored-by: João Almeida <eng-almeida@users.noreply.github.com>
Co-authored-by: Hanli Theron <HanliTheron@users.noreply.github.com>
Co-authored-by: John Brennan <brennj@users.noreply.github.com>
Co-authored-by: Dilvane Zanardine <dilvane@users.noreply.github.com>
Co-authored-by: Blake Johnston <johnstonbl01@users.noreply.github.com>
Co-authored-by: Zofia Korcz <Calanthe@users.noreply.github.com>
@brennj brennj deleted the copy-docs-build branch August 22, 2023 11:42
sandrina-p added a commit that referenced this pull request Nov 6, 2023
sandrina-p added a commit that referenced this pull request Nov 6, 2023
sandrina-p added a commit that referenced this pull request Nov 7, 2023
)

* fix: Remove stale attributes from conditionals

* Do not remove "type" while it is being deprecated

* realistic examples

* remove console.log

* Spot a bug, added a test to cover it. fix needed

* fix latest bug found. all good

* self-review

* self-review #2

* Release 0.7.2-dev.20231106093837

* delete unsused rootFieldAttrs param

* delete unneded default examples

* jsdcos remove x an y

* Revert back to 0.7.2-beta.0
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