Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update endpoint to use _ separator - Closes #6622 and #6666 #6760

Conversation

shuse2
Copy link
Collaborator

@shuse2 shuse2 commented Sep 10, 2021

What was the problem?

This PR resolves #6622 and resolves #6666

How was it solved?

  • Replace : with _
  • Replace _plugin_:loadingFinished with loaded unloaded and unloadedWithError.

How was it tested?

  • Updated unit tests
  • Use test app to check if plugin load/unload with child process. (This needed several code changes on APIClient and plugin, but since there are lots of build issue, it required to change the built code)

@shuse2 shuse2 self-assigned this Sep 10, 2021
@shuse2 shuse2 force-pushed the 6622-update_rpc_endpoint_naming branch from b09f51b to 4321a40 Compare September 10, 2021 08:25
Base automatically changed from 6707-refactor_controller to feature/6554-improve-framework-architecture September 10, 2021 09:27
@shuse2 shuse2 force-pushed the 6622-update_rpc_endpoint_naming branch from 4321a40 to 06944b6 Compare September 10, 2021 09:28
- Replace `:` with `_`
- Remove plugin lifecycle hooks
@shuse2 shuse2 force-pushed the 6622-update_rpc_endpoint_naming branch from 06944b6 to af21e9a Compare September 10, 2021 09:30
@shuse2 shuse2 force-pushed the 6622-update_rpc_endpoint_naming branch from 94c0cd8 to f69cae6 Compare September 10, 2021 11:46
@shuse2 shuse2 merged commit 244e9ef into feature/6554-improve-framework-architecture Sep 10, 2021
@shuse2 shuse2 deleted the 6622-update_rpc_endpoint_naming branch September 10, 2021 13:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants