Skip to content

docs(GreedyBear): add ASN aggregated feeds API#46

Merged
regulartim merged 3 commits intointelowlproject:mainfrom
drona-gyawali:api/asn
Jan 29, 2026
Merged

docs(GreedyBear): add ASN aggregated feeds API#46
regulartim merged 3 commits intointelowlproject:mainfrom
drona-gyawali:api/asn

Conversation

@drona-gyawali
Copy link
Copy Markdown
Contributor

@drona-gyawali drona-gyawali commented Jan 27, 2026

Added API documentation for the new ASN-aggregated feeds endpoint in GreedyBear.

Related PR:

cc: @regulartim @mlodic

Checklist

  • The pull request is for the branch main
  • If I added new documentation related to a new Pull Request that I created, I have added a reference to the related Pull Request
    • If the new documentation will be added in a future release, please add a reference like _Available from version > x.x.x_ where x.x.x is the current version number
  • If the additional documentation is a fix, you should directly push to the main branch and do not open this PR.

Signed-off-by: Dorna Raj Gyawali <dronarajgyawali@gmail.com>
@drona-gyawali
Copy link
Copy Markdown
Contributor Author

drona-gyawali commented Jan 27, 2026

I added a new serializer ASNFeedsOrderingSerializer in the GreedyBear code.
This serializer is internal and only used to validate ordering parameters for the ASN aggregated feeds endpoint.

Do you want me to include this serializer in the API documentation, or is documenting the endpoint enough?

@regulartim
Copy link
Copy Markdown
Collaborator

Do you want me to include this serializer in the API documentation, or is documenting the endpoint enough?

Documenting the endpoint is enough.

@regulartim
Copy link
Copy Markdown
Collaborator

You should however describe your endpoint on the usage page. Also, please add _Available from version >= 3.0.0_ there.

@drona-gyawali
Copy link
Copy Markdown
Contributor Author

Documenting the endpoint is enough.

Thanks, my bad I just noticed that we should mention it in USAGE.md. I’ll update that accordingly.

@drona-gyawali drona-gyawali marked this pull request as draft January 27, 2026 12:13
@drona-gyawali drona-gyawali marked this pull request as ready for review January 27, 2026 12:22
@regulartim regulartim merged commit 356f351 into intelowlproject:main Jan 29, 2026
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.

2 participants