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.

Sign up for GitHub

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

Jump to bottom

Endpoint: Get Comments from an Article #20

Open
mgroves opened this issue Jun 28, 2023 · 1 comment
Open

Endpoint: Get Comments from an Article #20

mgroves opened this issue Jun 28, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@mgroves
Copy link
Owner

mgroves commented Jun 28, 2023 •
edited
Loading

GET /api/articles/:slug/comments

Authentication optional, returns multiple comments
@mgroves
Copy link
Owner Author

mgroves commented Jun 28, 2023

Why is authentication optional? What does that mean, compared to "no authentication"?

@mgroves mgroves added the enhancement New feature or request label Jun 28, 2023
mgroves added a commit that referenced this issue Sep 22, 2023
@mgroves
#20 get comments, query, tests, migration
89268a9
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 trying compile flag for ScanConsistency in comments data Comments…
2445ab8 
…DataService
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 trying scanconsistency as option
a0b8182
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 trying to get more info why a couple of tests are failing only in CI
baf9899
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 trying again
6b50904
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 adding some error handling
5f7df54
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 trying to see if the root exception is in data service
53986e8
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20, ah I hardcoded bucket/scope name, that's probably it
af1179e
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 I screwed up, trying that again
fe9e5d7
mgroves added a commit that referenced this issue Sep 25, 2023
@mgroves
#20 I think it was an index issue, but turns out don't need an index
3bef149
mgroves added a commit that referenced this issue Sep 26, 2023
@mgroves
#20 updated docs
b2f44af
mgroves added a commit that referenced this issue Sep 26, 2023
@mgroves
#20 forget link to the spec in the docs comments
480f5f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mgroves