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

[Merged by Bors] - feat: Connectors beta reference docs #521

Closed
wants to merge 5 commits into from
Closed

[Merged by Bors] - feat: Connectors beta reference docs #521

wants to merge 5 commits into from

Conversation

tjtelan
Copy link
Contributor

@tjtelan tjtelan commented Mar 16, 2023

Preview updating connectors reference docs from their Hub package content using new fluvio-cms CLI tool.

The docs are rendered in a currently incomplete Connectors (beta) top-level section

Preview of top-nav w/ connectors side-nav
image

Example of one of the connector ref pages (HTTP, using its unaltered README)
image

tjtelan added 4 commits March 14, 2023 13:55
CLI will eventually manage content
This was using a manual process, fyi
Copies into connectors-beta section
Included updates to README with notes about `fluvio-cms` preview
@netlify
Copy link

netlify bot commented Mar 16, 2023

👷 Deploy request for fluvio-nightly pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit c77288e

Copy link
Contributor

@galibey galibey left a comment

Choose a reason for hiding this comment

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

LGTM

@sehz
Copy link
Contributor

sehz commented Mar 16, 2023

bors r+

bors bot pushed a commit that referenced this pull request Mar 16, 2023
Preview updating connectors reference docs from their Hub package content using new `fluvio-cms` CLI tool.

The docs are rendered in a **currently incomplete** `Connectors (beta)` top-level section

Preview of top-nav w/ connectors side-nav
![image](https://user-images.githubusercontent.com/344960/225605416-45b6febd-7a5c-44ab-b699-6e53a937bbac.png)

Example of one of the connector ref pages (HTTP, using its **unaltered** README)
![image](https://user-images.githubusercontent.com/344960/225605687-e1acc0db-28d3-4c2a-a78b-831101969004.png)
@bors
Copy link

bors bot commented Mar 16, 2023

Build failed:

@sehz
Copy link
Contributor

sehz commented Mar 16, 2023

@tjtelan can fix link? otherwise can't merge this

Just a workaround. Needs permanent solution
@tjtelan tjtelan marked this pull request as ready for review March 18, 2023 18:43
@sehz
Copy link
Contributor

sehz commented Mar 18, 2023

bors r+

bors bot pushed a commit that referenced this pull request Mar 18, 2023
Preview updating connectors reference docs from their Hub package content using new `fluvio-cms` CLI tool.

The docs are rendered in a **currently incomplete** `Connectors (beta)` top-level section

Preview of top-nav w/ connectors side-nav
![image](https://user-images.githubusercontent.com/344960/225605416-45b6febd-7a5c-44ab-b699-6e53a937bbac.png)

Example of one of the connector ref pages (HTTP, using its **unaltered** README)
![image](https://user-images.githubusercontent.com/344960/225605687-e1acc0db-28d3-4c2a-a78b-831101969004.png)
@bors
Copy link

bors bot commented Mar 18, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: Connectors beta reference docs [Merged by Bors] - feat: Connectors beta reference docs Mar 18, 2023
@bors bors bot closed this Mar 18, 2023
galibey added a commit that referenced this pull request Mar 31, 2023
* fix: Update cloud cli connector page (#496)

Resolves #487 

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Simple example for python smartmodule use (#500)

* bump versions (#514)

* Fix python (#512)



Co-authored-by: A.J <[email protected]>
Co-authored-by: simlay <[email protected]>

* feat: Connectors beta reference docs (#521)

Preview updating connectors reference docs from their Hub package content using new `fluvio-cms` CLI tool.

The docs are rendered in a **currently incomplete** `Connectors (beta)` top-level section

Preview of top-nav w/ connectors side-nav
![image](https://user-images.githubusercontent.com/344960/225605416-45b6febd-7a5c-44ab-b699-6e53a937bbac.png)

Example of one of the connector ref pages (HTTP, using its **unaltered** README)
![image](https://user-images.githubusercontent.com/344960/225605687-e1acc0db-28d3-4c2a-a78b-831101969004.png)

* Connectors (beta) Overview and first pass through cdk (#527)

Porting updates from #507

Also added hugo start stuff to fluvio-cms, deprecation notice to script

![image](https://user-images.githubusercontent.com/344960/226534225-72bc62c1-32e6-45c2-8764-7334f3724436.png)
![image](https://user-images.githubusercontent.com/344960/226534250-9bb3ffd6-8bbc-4a8a-b6f9-160333624b09.png)
![image](https://user-images.githubusercontent.com/344960/226534301-b59e0619-091b-4d07-9c00-0ad60a828a35.png)

CDK pages are still going through some work
![image](https://user-images.githubusercontent.com/344960/226534511-b0e003b8-1c21-48a1-9776-02f1d48aa6ca.png)

* chore: First complete pass through cdk docs (#529)

CDK docs have some narrative structure, but uncovered some possible friction worth fixing

* chore: Hide duplication for connectors docs (#532)

The old connector docs are hidden and have deprecation notices because they have a lot of current inter-site linking.

This also re-adds in the cloud connector page, with an update to the config file in the example. Another pass is needed to verify the tutorials. They should be reviewed to use the newer config and commands as well.

This should be the last of the changes made broken out from previously created PR #507 

Closes #507

* Updates related to connectors update (#536)

Adding link to requirements, so it is clear what that paragraph means

Co-authored-by: Luis Moreno <[email protected]>

* feat: update tutorial http to sql (#541)

Fixes #539

* sync connector's readme (#543)

* fix: Remove beta from connectors page (#544)

This is the only option for connectors, so we don't need the label

---------

Co-authored-by: T.J. Telan <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: simlay <[email protected]>
Co-authored-by: morenol <[email protected]>
Co-authored-by: A.J <[email protected]>
Co-authored-by: simlay <[email protected]>
Co-authored-by: morenol <[email protected]>
@tjtelan tjtelan deleted the connectors-beta branch November 16, 2023 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants