Skip to content

Feat/user preferences#110

Open
ajatdarojat45 wants to merge 9 commits into
choras-org:devfrom
ajatdarojat45:feat/user-preferences
Open

Feat/user preferences#110
ajatdarojat45 wants to merge 9 commits into
choras-org:devfrom
ajatdarojat45:feat/user-preferences

Conversation

@ajatdarojat45

Copy link
Copy Markdown
Contributor

Hi @SilvinWillemsen & @mberz,

This PR related to choras-org/frontend-v2#164 issue.

@SilvinWillemsen

Copy link
Copy Markdown
Contributor

Checked with choras-org/frontend-v2#167 and this works! @mberz Feel free to merge the frontend PR when this one is merged :)

@mberz mberz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your implementation. Can you please add numpy style docstrings to all functions and classes you added?

@ajatdarojat45

Copy link
Copy Markdown
Contributor Author

Well noted @mberz.

@ajatdarojat45

Copy link
Copy Markdown
Contributor Author

Hi @mberz,

I have added numpy style docstrings to all classes and functions i added.

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

Labels

None yet

Projects

Status: Require review

Development

Successfully merging this pull request may close these issues.

3 participants