Skip to content

#36 Documentation Issue resolved with addition of an example#55

Open
Vaibhav2154 wants to merge 3 commits intolayer5io:masterfrom
Vaibhav2154:master
Open

#36 Documentation Issue resolved with addition of an example#55
Vaibhav2154 wants to merge 3 commits intolayer5io:masterfrom
Vaibhav2154:master

Conversation

@Vaibhav2154
Copy link
Copy Markdown

Description

This PR Fixes #

Notes for Reviewers

Signed commits
Yes, I signed my commits.

Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
Signed-off-by: Vaibhav M N <vaibhavvaibhu2005@gmail.com>
@Vaibhav2154
Copy link
Copy Markdown
Author

@leecalcote Please review my pull request and let me know if there are any additonal changes to be made.

- Provide the desired configuration JSON through Meshery's UI or API.
- Deploy the filter and observe how it modifies JWT tokens based on your specifications.

For more details about configuring WASM filters with Meshery, visit the official ![Meshery documentation](https://github.com/meshery/.github/blob/master/profile/README.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, @Vaibhav2154.

The exclamation mark seems out of place here...

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Screenshot_20250102_050537
@leecalcote the redirecting link to Meshery Documentation is working fine. So I guess that '!' is in place correctly itself.
If there's anything to be resolved further for getting the PR merged please let me know.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Think harder on this, please.

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.

2 participants