Re: [Specifications] Add Use Case: Creating events indirectly (ie. with PUT) from Markus Lanthaler via GitHub on 2017-11-03 (public-hydra-logs@w3.org from November 2017)

From: Markus Lanthaler via GitHub <sysbot+gh@w3.org>
Date: Fri, 03 Nov 2017 07:31:56 +0000
To: public-hydra-logs@w3.org
Message-ID: <issue_comment.created-341636802-1509694315-sysbot+gh@w3.org>

@alien-mcl, please resolve the comments. Just LGTMing the PR isn't enough. You need the change the status of each comment:

![Resolve](http://postachio-images.s3.amazonaws.com/aa0e0e8e-5932-48c5-bbd5-bb782bc5caef/d0211a86-1b47-4614-80d9-50e7ea803076/91766e3d-05cf-41d2-a116-4dd5ddbe3be0.png)

---

Review status: all files reviewed at latest revision, 2 unresolved discussions.

---

*[drafts/use-cases/5.1.creating-event-with-put.md, line 34 at r1](https://reviewable.io:443/reviews/hydracg/specifications/143#-KxZXSL5Owpv1cqTG5AR:-Ky-lKgUksuo9_oWP_DE:b-giu4yq) ([raw file](https://github.com/hydracg/specifications/blob/f03e3028e6417adf2767a44c0c854b4af3b5fb4d/drafts/use-cases/5.1.creating-event-with-put.md#L34)):*
<details><summary><i>Previously, alien-mcl (Karol Szczepański) wrote…</i></summary><blockquote>

Looks nice - do you want to extend those operation arrays with fluent-like API? There may be already something that we could use.
</blockquote></details>

Yeah, that's the idea ;-)

---

*[drafts/use-cases/5.1.creating-event-with-put.md, line 72 at r1](https://reviewable.io:443/reviews/hydracg/specifications/143#-KxchCDgCDQQ4nzkMnf4:-Ky-mdhXQqs3nY9i9y23:b2nhswv) ([raw file](https://github.com/hydracg/specifications/blob/f03e3028e6417adf2767a44c0c854b4af3b5fb4d/drafts/use-cases/5.1.creating-event-with-put.md#L72)):*
<details><summary><i>Previously, alien-mcl (Karol Szczepański) wrote…</i></summary><blockquote>

I believe the spec doesn't say anything on where the value of the property being mapped should come from. In some cases indeed it can be taken from the payload, but for some other i.e. from user input or auto-generated. I think this is a topic for another story.
</blockquote></details>

Yep

---


*Comments from [Reviewable](https://reviewable.io:443/reviews/hydracg/specifications/143#-:-Ky-lQt7rvQL-bQMt-IJ:b-lr7h0v)*
<!-- Sent from Reviewable.io -->


-- 
GitHub Notification of comment by lanthaler
Please view or discuss this issue at https://github.com/HydraCG/Specifications/pull/143#issuecomment-341636802 using your GitHub account

Received on Friday, 3 November 2017 07:32:05 UTC