Skip to content

Conversation

@dkoutsonikolis
Copy link
Contributor

@dkoutsonikolis dkoutsonikolis commented Jun 30, 2021

Description

Add more levels in the Medical Personnel DB table.

Checklist

  • I have reviewed my code and checked that there are no unrelated changes in this pull request (old patches, accidental config files, etc)
  • I have created at least one test case for the changes I have made
  • I have updated the documentation (SwaggerHub, README.md, Wiki, etc.) for the changes I have made

@TDaglis TDaglis temporarily deployed to tmh-registry-feat-add-m-ugm6vv June 30, 2021 14:35 Inactive

def __str__(self):
return self.user.username
return f"({self.level}) {self.user.username}"
Copy link
Contributor

Choose a reason for hiding this comment

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

@dkoutsonikolis What about self.get_level_display() instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed it - not sure if there is a better way.

Copy link
Contributor Author

@dkoutsonikolis dkoutsonikolis Jun 30, 2021

Choose a reason for hiding this comment

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

Never mind, all good. :)

@TDaglis TDaglis temporarily deployed to tmh-registry-feat-add-m-ugm6vv June 30, 2021 14:59 Inactive
@TDaglis TDaglis temporarily deployed to tmh-registry-feat-add-m-ugm6vv June 30, 2021 15:06 Inactive
Copy link
Contributor

@dpapaspyros dpapaspyros left a comment

Choose a reason for hiding this comment

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

LGTM!!

@dkoutsonikolis dkoutsonikolis merged commit ebc8e6b into main Jul 1, 2021
@dkoutsonikolis dkoutsonikolis deleted the feat/add-medical-personnel-levels branch July 1, 2021 06:24
@github-actions
Copy link

github-actions bot commented Jul 1, 2021

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants