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 AddSetCookies instance for when the left tree is the same before and after the transformation #1698

Closed

Conversation

ohri-anurag
Copy link
Contributor

Adding code suggested by @mastarija on issue #1601

@ohri-anurag
Copy link
Contributor Author

The CI fails here, but this is what I did on my machine:

  1. Modified this line in shell.nix:
-      buildInputs = [ ghc zlib python3 wget cabal-install postgresql openssl stack haskellPackages.hspec-discover ]
+      buildInputs = [ ghc zlib python3 wget cabal-install postgresql openssl stack haskellPackages.hspec-discover haskellPackages.doctest ]
  1. Ran this command: nix-shell nix/shell.nix --argstr compiler ghc865Binary
  2. Both cabal build all and cabal test all passed in the shell entered at step 2.
  3. I have also tested this with ghc 9.0.1, without any changes to shell.nix, it works.

@ysangkok
Copy link
Contributor

Could you please rebase on master?

worm2fed and others added 4 commits August 21, 2023 10:21
* Use `Header'` in response headers

Use `Header'` instead of `Header` in response, so it's possible to provide
`Description`, for example:

```
type PaginationTotalCountHeader =
  Header'
    '[ Description "Indicates to the client total count of items in collection"
     , Optional
     , Strict
     ]
    "Total-Count"
    Int
```

Note: if you want to add header with description you should use `addHeader'`
or `noHeader'` which accepts `Header'` with all modifiers.
@ysangkok
Copy link
Contributor

I don't think this rebase was done right, since some commits from the master branch are shown here.

@ohri-anurag ohri-anurag deleted the add-AddSetCookies-instance branch August 22, 2023 04:13
@ohri-anurag
Copy link
Contributor Author

Closed in favor of #1702

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.

4 participants