Skip to content

Implementing getCurrentInstant() authoring method#5

Merged
julioalex-rezende merged 2 commits intomasterfrom
julio/getCurrentInstant
Jun 13, 2023
Merged

Implementing getCurrentInstant() authoring method#5
julioalex-rezende merged 2 commits intomasterfrom
julio/getCurrentInstant

Conversation

@julioalex-rezende
Copy link

@julioalex-rezende julioalex-rezende commented Jun 12, 2023

Description

Exposing getCurrentInstant( ) method for Workflow Authoring API.
Included API method call to tests to ensure its functionality.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #TASK 46604

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Copy link
Collaborator

@macromania macromania left a comment

Choose a reason for hiding this comment

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

LGTM 🌝

Copy link

@Aymalla Aymalla left a comment

Choose a reason for hiding this comment

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

LGTM, Just has a minor comment

@julioalex-rezende julioalex-rezende force-pushed the julio/getCurrentInstant branch from 9dcc4d2 to 3fa207d Compare June 13, 2023 13:40
@julioalex-rezende julioalex-rezende merged commit 4b26a79 into master Jun 13, 2023
macromania pushed a commit that referenced this pull request Jun 21, 2023
Co-authored-by: Julio Rezende <jsilvarezend@microsoft.com>
Signed-off-by: Mahmut Canga <cangamahmut@gmail.com>
Signed-off-by: Julio Rezende <jsilvarezend@microsoft.com>
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.

3 participants