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

archive kafka events plugin #1423

Merged
merged 3 commits into from
Jan 15, 2025
Merged

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Jan 15, 2025

Closes #1422.

@dbluhm dbluhm requested a review from swcurran January 15, 2025 20:05
@dbluhm
Copy link
Contributor Author

dbluhm commented Jan 15, 2025

Perhaps we should include a brief description of the plugin?

@swcurran
Copy link
Contributor

Yes — a description of the archived plugin would be helpful.

Sorry about my answer on how to add the document. I had been thinking you were in the ACA-Py repo. In this repo, the mkdocs hierarchy is pretty much all generated, so putting where you did is the right thing.

swcurran
swcurran previously approved these changes Jan 15, 2025
Copy link
Contributor

@swcurran swcurran left a comment

Choose a reason for hiding this comment

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

Can merge now and update with the second part or I’m happy to re-review this one.

@dbluhm
Copy link
Contributor Author

dbluhm commented Jan 15, 2025

@swcurran done; I'll let you do another pass

@dbluhm dbluhm changed the title docs: add archived plugins doc archive kafka events plugin Jan 15, 2025
@swcurran swcurran merged commit e1db379 into openwallet-foundation:main Jan 15, 2025
4 checks passed
@swcurran
Copy link
Contributor

DId you fix the settings? I checked and new commits should auto-dismiss reviews...

@dbluhm
Copy link
Contributor Author

dbluhm commented Jan 15, 2025

Yeah, I did a quick tweak of the settings

@dbluhm dbluhm deleted the docs/archived branch January 15, 2025 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants