Skip to content

API joint for pulsar backend: Get List of function, Get function info, Get function stats, Get function status - #25

Merged
RobertIndie merged 9 commits into
FunctionStream:mainfrom
Akhil-Yuan:main
Jun 18, 2021
Merged

RobertIndie merged 9 commits into
FunctionStream:mainfrom
Akhil-Yuan:main

Conversation

@Akhil-Yuan

@Akhil-Yuan Akhil-Yuan commented Jun 7, 2021 •

Copy link
Copy Markdown
Contributor

Master Issue: #8

@RobertIndie
RobertIndie requested a review from yezhidaizhu June 7, 2021 10:37
@RobertIndie RobertIndie added this to the 0.1.0 milestone Jun 7, 2021

@RobertIndie RobertIndie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The incorrect format URL caused the failure to get the function status.

Comment thread front-end/.env.development Outdated
Comment thread front-end/babel.config.js Outdated
Comment thread front-end/src/api/func.js Outdated
Comment thread front-end/src/utils/request.js Outdated
@Akhil-Yuan
Akhil-Yuan requested review from RobertIndie and removed request for yezhidaizhu June 8, 2021 10:30

@RobertIndie RobertIndie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think there are still some problems with this PR. I got a 404 error when I open the index page.
In #26, I have set up an API env for this PR. You could merge the main branch to this PR then you can get that env. Also, you need to pay attention to modify the base URL.

@Akhil-Yuan
Akhil-Yuan requested a review from RobertIndie June 13, 2021 06:15

@RobertIndie RobertIndie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍 Overall looks good to me!
We also need to push the changes of the .env.api here.

@RobertIndie RobertIndie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@Akhil-Yuan
Akhil-Yuan requested a review from RobertIndie June 16, 2021 12:34
@RobertIndie

Copy link
Copy Markdown
Contributor

Getting close! 👍 Please resolve conflicts

@RobertIndie RobertIndie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@RobertIndie
RobertIndie merged commit a521393 into FunctionStream:main Jun 18, 2021
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