docs: retrieve user by ID #5044

datapumpernickel · 2024-06-18T09:28:49Z

Description

Retrieving a user by ID works only when passing in an ID of class UUID. This has been updated in the docs, as well as in the function examples and one error message of the users.py functions.

Closes #5011

Documentation update

How Has This Been Tested

Not tested yet.

Checklist

I added relevant documentation
I followed the style guidelines of this project
I did a self-review of my code
I made corresponding changes to the documentation
My changes generate no new warnings
I filled out the contributor form (see text above)
I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

…illa-io#5011

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

sdiazlor · 2024-06-19T07:05:35Z

Thanks @datapumpernickel, could you change the branch to main?

davidberenstein1957 · 2024-06-19T07:55:01Z

@datapumpernickel, would you be able to change the code so we can obtain users either by str or UUID?

datapumpernickel · 2024-06-19T07:57:59Z

Woopsie, I was trying to change this to the main branch and did not see your reply @davidberenstein1957 in the meantime. Sorry about the mess.
I could look at it, probably not this week, as I will be off until Mid-July. But then I can give it a try!

davidberenstein1957 · 2024-06-19T08:03:58Z

Hi @datapumpernickel, no problem. Your contribution is great. I think it might be nicer for users to be able to pass either an id or UUID. @frascuchon @burtenshaw WDYT? Either way, Enjoy the holidays :)

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

datapumpernickel · 2024-06-19T08:14:52Z

Definitely more convenient to be able to pass both -thanks for the hint. It actually seemed pretty straight forward, to I just put in a commit with a small change that checks if the ID is of instance uuid.UUID, if not, it tries to convert it to an instance of uuid.UUID. It returns an error if that fails for some reason.

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

davidberenstein1957 · 2024-10-17T07:35:50Z

@datapumpernickel , I realised this had become stale. Sorry for that. Would you be able to resolve the merge conflict? Also, I think the client.datasets and client.workspaces have similar methods that need to be tackled.

…rgilla-io#5644) # Description Following the discussion at https://discord.com/channels/879548962464493619/1300627522135916604 we have removed unnecessary exposed ports at our Docker compose file, increasing the security. **Type of change** - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** - [x] Running docker compose manually. **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

@leiyre

…o#5647) # Description  This change will allow at least to run the frontend build pipeline. Currently, we have no permissions to deploy on gcp from GitHub actions, so the PR preview is not available. cc @leiyre @damianpumar @Amelie-V @nataliaElv **Type of change**  **How Has This Been Tested**  **Checklist**  - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

…into releases/2.4.0

# Description  Closes #<issue_number> **Type of change**  - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) - Documentation update **How Has This Been Tested**  **Checklist**  - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

# Description Adding some updates for our `CHANGELOG.md` for release `2.4.0` **Type of change** - Documentation update **How Has This Been Tested** - [x] Rendering markdowns 😄 **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: Paco Aranda <[email protected]> Co-authored-by: Francisco Aranda <[email protected]>

…file (argilla-io#5651) # Description Small improvement moving the install of some dependencies before cleaning up. **Type of change** - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** - [ ] Deploying it to a new HF space. **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

…into releases/2.4.0

…not available (argilla-io#5654) # Description This should fix errors raised by our pre-commit actions when trying to run `helm lint` and the `helm` command is not available. I have tried to add `helm` as additional dependency for the `helmlint` hook but is not possible so disabling it by now on CI. It is available when running it locally and you have `helm` installed. **Type of change** - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** - [ ] GitHub Actions with pre-commit should now work as expected without errors. **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

# Description  Closes #<issue_number> **Type of change**  - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - Refactor (change restructuring the codebase without changing functionality) - Improvement (change adding some improvement to an existing functionality) - Documentation update **How Has This Been Tested**  **Checklist**  - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/) --------- Co-authored-by: Leire Aguirre <[email protected]> Co-authored-by: José Francisco Calvo <[email protected]> Co-authored-by: José Francisco Calvo <[email protected]> Co-authored-by: Paco Aranda <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Damián Pumar <[email protected]> Co-authored-by: Francisco Aranda <[email protected]> Co-authored-by: burtenshaw <[email protected]>

datapumpernickel · 2024-11-03T22:17:20Z

Hi @davidberenstein1957, sorry, I was not able to look at this in the past weeks. I just wanted to resolve the conflicts but upon testing it again, I realised that somehow this error has been patched:

This works now. 🎊 🚀

retrieved_user = client.users(id="0d05d426-294e-4524-b391-3d8db415537c")
retrieved_user

All the best for the upcoming release and thanks again!

davidberenstein1957 · 2024-11-04T10:17:22Z

@datapumpernickel, no problem. Thanks for the effort. Hope to be able to follow up a bit better with your next PR. Take care :)

datapumpernickel added 2 commits June 18, 2024 11:22

docs: updated documentation to use UUID class for retrieving user arg…

08384a0

…illa-io#5011

docs: updated examples in function for retrieving user argilla-io#5011

9a158c7

datapumpernickel changed the title ~~Documentation update~~ docs: retrieve user by ID Jun 18, 2024

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

15f65f5

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

datapumpernickel mentioned this pull request Jun 18, 2024

[DOCS] pulling user by ID does not work as specified in the docs #5011

Closed

datapumpernickel changed the base branch from develop to main June 19, 2024 07:49

davidberenstein1957 changed the base branch from main to develop June 19, 2024 07:53

davidberenstein1957 requested a review from sdiazlor June 19, 2024 07:54

datapumpernickel changed the base branch from develop to main June 19, 2024 07:55

datapumpernickel closed this Jun 19, 2024

datapumpernickel reopened this Jun 19, 2024

datapumpernickel changed the base branch from main to develop June 19, 2024 07:56

datapumpernickel and others added 2 commits June 19, 2024 10:12

convert str id to UUID instance, if not already of instance uuid.UUID

76a9808

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

7d942b6

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

datapumpernickel and others added 2 commits June 19, 2024 10:18

fix typo in except block

19f3c16

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

a77b125

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

davidberenstein1957 added this to the v2.4.0 milestone Oct 17, 2024

frascuchon and others added 4 commits October 29, 2024 10:53

chore: Set release version

d390a02

fix and update test snaps

53fd172

clean code

9ef7c9c

frascuchon modified the milestones: v2.4.0, v2.5.0 Oct 29, 2024

fix lint

e8ab4df

frascuchon and others added 21 commits October 29, 2024 13:32

chore: Fix missing str format

c96a815

fix: Import missing Argilla component

94ff316

Dataset creation feature final QA (argilla-io#5646)

e81b37c

remove focus for dataset list

f86859d

Merge branch 'releases/2.4.0' of https://github.com/argilla-io/argilla …

000efec

…into releases/2.4.0

qa

bab0890

update styles

8228d41

increase dataset creation retries to 30

0494dee

update dataset creation retries to 10

d18b79d

chore: small code style change

3f99cdb

update styles and copy

c2dddc3

fix separator

c5b54ca

Merge branch 'releases/2.4.0' of https://github.com/argilla-io/argilla …

c060372

…into releases/2.4.0

include id

86fb672

Merge remote-tracking branch 'upstream/main' into develop

d7a39e5

datapumpernickel closed this Nov 3, 2024

davidberenstein1957 removed this from the v2.5.0 milestone Nov 26, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs: retrieve user by ID #5044

docs: retrieve user by ID #5044

datapumpernickel commented Jun 18, 2024

sdiazlor commented Jun 19, 2024

davidberenstein1957 commented Jun 19, 2024

datapumpernickel commented Jun 19, 2024 •

edited

Loading

davidberenstein1957 commented Jun 19, 2024 •

edited

Loading

datapumpernickel commented Jun 19, 2024

davidberenstein1957 commented Oct 17, 2024

datapumpernickel commented Nov 3, 2024

davidberenstein1957 commented Nov 4, 2024

docs: retrieve user by ID #5044

docs: retrieve user by ID #5044

Conversation

datapumpernickel commented Jun 18, 2024

Description

sdiazlor commented Jun 19, 2024

davidberenstein1957 commented Jun 19, 2024

datapumpernickel commented Jun 19, 2024 • edited Loading

davidberenstein1957 commented Jun 19, 2024 • edited Loading

datapumpernickel commented Jun 19, 2024

davidberenstein1957 commented Oct 17, 2024

datapumpernickel commented Nov 3, 2024

davidberenstein1957 commented Nov 4, 2024

datapumpernickel commented Jun 19, 2024 •

edited

Loading

davidberenstein1957 commented Jun 19, 2024 •

edited

Loading