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

uuid in read project and total_task in generate-log #978

Merged
merged 4 commits into from
Nov 21, 2023

Conversation

Sujanadh
Copy link
Collaborator

@Sujanadh Sujanadh commented Nov 7, 2023

added uuid in read project and total task count in generate log

@spwoodcock
Copy link
Member

spwoodcock commented Nov 7, 2023

What is the uuid used for? @Sujanadh

It's not saved to the database, so it will be different every time you call the API - is that intended?

@spwoodcock
Copy link
Member

@Sujanadh just wondering if I should merge this? What is the use case for the uuid?

@nrjadkry
Copy link
Member

nrjadkry commented Nov 9, 2023

@spwoodcock this was intended for displaying the project creation log in the project detail page. But we might need to revise it if we use apscheduler. So, let's not merge it yet until we finalize #977

@spwoodcock
Copy link
Member

Is this feature still planned?

And should we merge this now if we don't use apscheduler?

@nrjadkry
Copy link
Member

@spwoodcock , we can merge this now. It should work fine now since we won't be using apscheduler

@spwoodcock spwoodcock merged commit 9281216 into development Nov 21, 2023
5 of 6 checks passed
@spwoodcock spwoodcock deleted the uuid_in_read_project branch November 21, 2023 11:14
nischalstha9 pushed a commit to naxa-developers/fmtm that referenced this pull request Dec 8, 2023
)

* feat: added uuid on read projects

* feat: added total task count in generate-log

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* refactor: set uuid directly for project_uuid

---------

Co-authored-by: sujanadh <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: spwoodcock <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants