Skip to content

Conversation

@kollil
Copy link
Collaborator

@kollil kollil commented May 11, 2021

Rationale

This update contains the PMIC flow after the taskid is created. A new label is created for PMIC task to "More actions" list.

@kollil kollil requested a review from labkey-jeckels May 11, 2021 17:22
Copy link
Contributor

@labkey-jeckels labkey-jeckels left a comment

Choose a reason for hiding this comment

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

This looks fine if you want to proceed as-is, but I know you're interested in splitting out the query so that you can avoid having two Create Task menu items

@kollil kollil merged commit b0260e8 into release20.11-SNAPSHOT May 17, 2021
@kollil kollil deleted the 20.11_fb_Changes_PMIC branch May 17, 2021 23:55
@kollil
Copy link
Collaborator Author

kollil commented May 18, 2021

Josh, After I merged the changes, I see an error on team city for the build #224. That update is also included in the final merge.
Does that effect my merge?

@labkey-ankurj
Copy link
Contributor

Hi @kollil , Josh is out this week, think that error is not related to any change. After getting merged, there is a new build running on the default branch https://teamcity.labkey.org/buildConfiguration/LabKey_2011Release_External_Onprc_OnprcInstaller?branch=%3Cdefault%3E&buildTypeTab=overview&mode=builds

@kollil
Copy link
Collaborator Author

kollil commented May 18, 2021

Thank you Ankur for checking!

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.

4 participants