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

Use available_channel_type table data to create channelForm #143

Merged
merged 9 commits into from
Dec 19, 2023

Conversation

sukhwinder33445
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 commented Oct 24, 2023

@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Oct 24, 2023
@sukhwinder33445 sukhwinder33445 marked this pull request as draft October 24, 2023 11:28
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 5 times, most recently from 370f7fd to f44f842 Compare October 25, 2023 11:49
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch from f44f842 to 6c3c99a Compare October 26, 2023 11:22
@sukhwinder33445 sukhwinder33445 self-assigned this Oct 26, 2023
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 2 times, most recently from e3c607e to bc26fa9 Compare October 30, 2023 10:56
@sukhwinder33445 sukhwinder33445 changed the title Use plugin table data to create channelForm Use available_channel_type table data to create channelForm Oct 30, 2023
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 3 times, most recently from f97ad40 to c3cf9ae Compare October 31, 2023 14:18
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 2 times, most recently from 1eb205c to 762856e Compare November 8, 2023 11:48
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 2 times, most recently from 37fd34c to c277075 Compare November 20, 2023 15:28
@nilmerg nilmerg marked this pull request as ready for review November 28, 2023 08:05
Copy link
Member

@nilmerg nilmerg left a comment

Choose a reason for hiding this comment

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

Please rebase and fix static analysis errors.

library/Notifications/Model/Channel.php Outdated Show resolved Hide resolved
library/Notifications/Model/Channel.php Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
@nilmerg nilmerg added this to the Preview #1 milestone Nov 30, 2023
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch from 6377689 to ecdf7bd Compare December 1, 2023 10:24
- A channel plugin now defines required config elements and insert them into plugin table's `config_attrs` column as json
As the plugins provide there own config elements, hardcoded check for rocketchat and email isn't ideal.
But exceptionally, the email check remains as it is the most used at the moment.
Multiple channels can be defined with same type, so the channel name is more relevant.
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch 2 times, most recently from 063dca4 to ece0dc2 Compare December 4, 2023 09:51
@sukhwinder33445 sukhwinder33445 force-pushed the create-channel-form-using-plugin-table branch from ece0dc2 to 26601ae Compare December 4, 2023 10:54
application/forms/ChannelForm.php Outdated Show resolved Hide resolved
@nilmerg nilmerg merged commit 8506590 into main Dec 19, 2023
13 checks passed
@nilmerg nilmerg deleted the create-channel-form-using-plugin-table branch December 19, 2023 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants