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

[blog] added blog post on REST API #123

Merged
merged 7 commits into from
May 9, 2024
Merged

[blog] added blog post on REST API #123

merged 7 commits into from
May 9, 2024

Conversation

PrajwolAmatya
Copy link
Contributor

Description

Added blog post on REST API.

@PrajwolAmatya PrajwolAmatya added the blog New blog label Apr 15, 2024
@PrajwolAmatya PrajwolAmatya self-assigned this Apr 15, 2024
@PrajwolAmatya PrajwolAmatya force-pushed the blog/rest-api branch 2 times, most recently from 8fc8e60 to c9cb323 Compare April 16, 2024 03:39
@PrajwolAmatya PrajwolAmatya marked this pull request as ready for review April 16, 2024 03:40
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
src/assets/RestApi/rest-api.md Show resolved Hide resolved
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
@S-Panta
Copy link

S-Panta commented May 8, 2024

LGTM

S-Panta
S-Panta previously approved these changes May 8, 2024
koebel
koebel previously approved these changes May 8, 2024
Copy link
Contributor

@koebel koebel left a comment

Choose a reason for hiding this comment

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

LGTM, just update the date to the actual date before publishing

src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
SagarGi
SagarGi previously approved these changes May 8, 2024
Copy link
Contributor

@SagarGi SagarGi left a comment

Choose a reason for hiding this comment

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

LVGTM 👍

src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
koebel
koebel previously approved these changes May 8, 2024
Copy link
Contributor

@koebel koebel left a comment

Choose a reason for hiding this comment

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

LGTM

src/assets/RestApi/rest-api.md Outdated Show resolved Hide resolved
@nabim777
Copy link
Member

nabim777 commented May 9, 2024

just a minor spelling mistake and a missing comma. Apart from that,
LGTM 👍

Copy link
Contributor

@koebel koebel left a comment

Choose a reason for hiding this comment

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

good that you caught some more small spelling mistakes

@PrajwolAmatya PrajwolAmatya merged commit 6021c48 into master May 9, 2024
1 check passed
@PrajwolAmatya PrajwolAmatya deleted the blog/rest-api branch May 9, 2024 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blog New blog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants