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 CouchDB adapter specification #542

Merged
merged 3 commits into from
Oct 24, 2019
Merged

Conversation

lionelvillard
Copy link
Contributor

No description provided.

Signed-off-by: Lionel Villard <villard@us.ibm.com>
| :-------------------- | :--------------------------------------- |
| `id` | The event sequence identifier (`seq`) |
| `source` | The server URL / db_name |
| `specversion` | `1.0-rc1` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's be optimistic and make this 1.0 :-)

Copy link
Contributor

@JemDay JemDay left a comment

Choose a reason for hiding this comment

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

In general i think we should remove any of the items that have a value of "omit"

| `specversion` | `1.0` |
| `type` | `org.apache.couchdb.document.update` |
| `datacontenttype` | `application/json` |
| `dataschema` | Omit |
Copy link
Contributor

Choose a reason for hiding this comment

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

If we're omitting this then i'd suggest removing the attribute completely.

| `type` | `org.apache.couchdb.document.update` |
| `datacontenttype` | `application/json` |
| `dataschema` | Omit |
| `subject` | `id` value |
Copy link
Contributor

Choose a reason for hiding this comment

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

I assume this is the "Id" of the document that changed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes. Updating...

Each section below describes how to determine the CloudEvents attributes
based on the event type.

## /db/_changes
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the _ makes everything else after this show up in italics. I think if you escape it (e.g. \_) then it will fix that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

Signed-off-by: Lionel Villard <villard@us.ibm.com>
Signed-off-by: Lionel Villard <villard@us.ibm.com>
@duglin
Copy link
Collaborator

duglin commented Oct 24, 2019

Approved on 10/24 call

@duglin duglin merged commit f64d796 into cloudevents:master Oct 24, 2019
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.

3 participants