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] - Connectors (beta) Overview and first pass through cdk #527

Closed
wants to merge 3 commits into from
Closed

[Merged by Bors] - Connectors (beta) Overview and first pass through cdk #527

wants to merge 3 commits into from

Conversation

tjtelan
Copy link
Contributor

@tjtelan tjtelan commented Mar 21, 2023

Porting updates from #507

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

image
image
image

CDK pages are still going through some work
image

Overview and first pass through cdk

Also added hugo start stuff to fluvio-cms, deprecation notice to script
@netlify
Copy link

netlify bot commented Mar 21, 2023

👷 Deploy request for fluvio-nightly pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 75f02c6

Copy link
Contributor

@sehz sehz left a comment

Choose a reason for hiding this comment

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

LGTM. we can merge this and iterate

Just a simple pass with copied content
@tjtelan tjtelan marked this pull request as ready for review March 22, 2023 02:55
@tjtelan
Copy link
Contributor Author

tjtelan commented Mar 22, 2023

bors r+

@bors
Copy link

bors bot commented Mar 22, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title Connectors (beta) Overview and first pass through cdk [Merged by Bors] - Connectors (beta) Overview and first pass through cdk Mar 22, 2023
@bors bors bot closed this Mar 22, 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-overview 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