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] - chore: Hide duplication for connectors docs #532

Closed
wants to merge 14 commits into from
Closed

[Merged by Bors] - chore: Hide duplication for connectors docs #532

wants to merge 14 commits into from

Conversation

tjtelan
Copy link
Contributor

@tjtelan tjtelan commented Mar 25, 2023

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

@netlify
Copy link

netlify bot commented Mar 25, 2023

👷 Deploy request for fluvio-nightly pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit a23f274

@github-actions github-actions bot added the CI label Mar 25, 2023
@tjtelan
Copy link
Contributor Author

tjtelan commented Mar 25, 2023

bors r+

bors bot pushed a commit that referenced this pull request Mar 25, 2023
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
@bors
Copy link

bors bot commented Mar 25, 2023

Build failed:

@tjtelan
Copy link
Contributor Author

tjtelan commented Mar 25, 2023

bors r-

Not sure if actions will build site properly...

tjtelan added 6 commits March 25, 2023 12:10
Didn't get caught locally with automated tests...
Not sure why this is only caught in CI and not locally
@tjtelan
Copy link
Contributor Author

tjtelan commented Mar 25, 2023

bors r+

bors bot pushed a commit that referenced this pull request Mar 25, 2023
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
@bors
Copy link

bors bot commented Mar 25, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore: Hide duplication for connectors docs [Merged by Bors] - chore: Hide duplication for connectors docs Mar 25, 2023
@bors bors bot closed this Mar 25, 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-deprecate branch November 16, 2023 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment