Skip to content

Remove restricted types as not stubs job to validate API calls. #3

Description

@rupertlssmith

There is an issue with refined strings not matching their regexes correctly. This could probably be solved by escaping the regex correctly. However, after discussing it, decided it would be better to not do this field-by-field validation of requests and responses. It makes the API awkward to use, and is unusual for a client stub to do this anyway, since the AWS services themselves validate their inputs.

Remove the refined Int and String types, and replace with plain Ints and Strings.

To achieve this:

  1. Add options to the Elm.Lang code generation module in Salix. It should be an L3 processor.
  2. Use the same options in the Elm.Json.Coding module.
  3. Make these options common to the above processors.

By doing the above, the use of refined types becomes a parameter of the code generator - can put it back at a later date if desired. Or possible validate responses, but not requests, etc.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions