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

add fuzzy match search example to custom query docs #1555

Merged
merged 3 commits into from
Feb 28, 2019

Conversation

nathanwaters
Copy link
Contributor

Description

Fuzzy match search example added to custom SQL functions docs.

What component does this PR affect?

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Requires changes from other components? If yes, please mark the components:

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Related Issue

Solution and Design

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update
  • Community content

Checklist:

  • I have read the contributing guide and my code conforms to the guidelines.
  • This change requires a change in the documentation.
  • I have updated the documentation accordingly.
  • I have added required tests.

@hasura-bot
Copy link
Contributor

Beep boop! 🤖

Hey @nathanwaters, thanks for your PR!

One of my human friends will review this PR and get back to you as soon as possible. 🕐

Stay awesome! 😎

@CLAassistant
Copy link

CLAassistant commented Feb 4, 2019

CLA assistant check
All committers have signed the CLA.

@shahidhk shahidhk requested a review from rikinsk-zz February 4, 2019 07:21
@shahidhk shahidhk added the c/docs Related to docs label Feb 4, 2019
@hasura-bot
Copy link
Contributor

Review app for commit 9362151 deployed to Heroku: https://hge-ci-pull-1555.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1555-9362151

@shahidhk shahidhk self-assigned this Feb 4, 2019
@rikinsk-zz rikinsk-zz merged commit 9cfdef3 into hasura:master Feb 28, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-1555.herokuapp.com is deleted

@hasura-bot
Copy link
Contributor

Beep boop! 🤖

Whoa! 🎉 🎉 💃

GIF

Awesome work @nathanwaters! 💪 🏆 All of us at Hasura ❤️ what you did.

Thanks again 🤗

@hasura-bot
Copy link
Contributor

Review app for commit 526f5a9 deployed to Heroku: https://hge-ci-pull-1555.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1555-526f5a9

hasura-bot pushed a commit that referenced this pull request Jan 22, 2025
This PR adds support for configuring and handling response headers for
pre-route plugins.

### What

Added support for response header configuration in lifecycle plugin
hooks, enabling:
- Header forwarding from plugin responses
- Custom header injection

### How

1. Added response configuration support:
   - New `response` field in plugin hook config structures
   - Updated JSON schemas in both MBS and OpenDD

2. Implemented response handling:
   - Modified `PreRoutePluginResponse` to include headers
   - Added header processing in plugin execution flow

Example configuration:
```yaml
kind: LifecyclePluginHook
version: v1
definition:
  pre: route
  name: restified_endpoints
  url:
    value: http://localhost:5001/restified
  config:
    match: "/v1/api/rest/*"
    matchMethods: ["GET", "POST"]
    request:
      method: POST
      headers:
        forward:
          - Authorization
        additional:
          hasura-m-auth:
            value: "your-strong-m-auth-key"
      rawRequest:
        path: {}
        query: {}
        method: {}
        body: {}
    response:
      headers:
        forward:
          - content-type
          - x-request-id
        additional:
          x-processed-by:
            value: "restified-plugin"
```
V3_GIT_ORIGIN_REV_ID: d994c1070bc37ab796b0512dd20c5c4135712441
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/docs Related to docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants