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

workspace provider is updated #584

Merged
merged 1 commit into from
Dec 7, 2023
Merged

workspace provider is updated #584

merged 1 commit into from
Dec 7, 2023

Conversation

Zulalm
Copy link
Contributor

@Zulalm Zulalm commented Dec 7, 2023

Request sending mechanism of the workspace provider is changed to handle the post requests. Now, it works as expected.

@Zulalm Zulalm added difficulty: moderate Requires more than basic skills and knowledge, and may require research or collaboration to complete type: fix This label should be applied to issues and pull requests that contain a fix . frontend labels Dec 7, 2023
@Zulalm Zulalm self-assigned this Dec 7, 2023
mehmetSuzer
mehmetSuzer approved these changes Dec 7, 2023
View reviewed changes
Copy link
Contributor

@mehmetSuzer mehmetSuzer left a comment

Choose a reason for hiding this comment

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

Good Job!

@mehmetSuzer mehmetSuzer merged commit a3b8aea into frontend Dec 7, 2023
@mehmetSuzer mehmetSuzer deleted the 583-fe-fix-workspace-provider branch December 7, 2023 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mehmetSuzer mehmetSuzer mehmetSuzer approved these changes

Assignees

@Zulalm Zulalm

Labels
difficulty: moderate Requires more than basic skills and knowledge, and may require research or collaboration to complete frontend type: fix This label should be applied to issues and pull requests that contain a fix .
Projects
None yet
Milestone
CMPE451 Final Milestone: Presentation...
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Zulalm @mehmetSuzer