Skip to content

Fixes #1401#1402

Closed
balbany wants to merge 1 commit intopnp:devfrom
balbany:issue-1401
Closed

Fixes #1401#1402
balbany wants to merge 1 commit intopnp:devfrom
balbany:issue-1401

Conversation

@balbany
Copy link
Copy Markdown
Contributor

@balbany balbany commented Mar 10, 2020

Fixes #1401
isValidTeamsChannelId - Fixing regex to allow 'tacv2' channel id format.

allow 'tacv2' channel id format.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 01d14da on balbany:issue-1401 into ee3c531 on pnp:dev.

@balbany
Copy link
Copy Markdown
Contributor Author

balbany commented Mar 10, 2020

Any chance that a super-small bugfix like this could be merged quickly? I'd rather not have to manually install my fork accross multiple servers... TIA!

@VelinGeorgiev
Copy link
Copy Markdown
Contributor

@balbany, we can try to merge it with the next beta, that means next Sunday. Does that suit your timings?

@balbany
Copy link
Copy Markdown
Contributor Author

balbany commented Mar 10, 2020

Sunday the 15th? That should be fine, thanks

@VelinGeorgiev
Copy link
Copy Markdown
Contributor

Thanks! Adding @waldekmastykarz , @garrytrinder so they know this must go on the fast track.

Copy link
Copy Markdown
Contributor

@VelinGeorgiev VelinGeorgiev left a comment

Choose a reason for hiding this comment

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

Looks good @balbany ! Thank you very much!

@waldekmastykarz
Copy link
Copy Markdown
Member

Merged manually. Thank you @balbany 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants