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.

Sign up for GitHub

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

Jump to bottom

replace use of `String` to store IDs with `Ulid` #1125

Open

de-sh opened this issue Jan 20, 2025 · 2 comments

Assignees

Labels

enhancement good first issue

Contributor

de-sh commented Jan 20, 2025 •

edited

Loading

parseable/src/correlation.rs

Line 152 in 6c5f2aa

pub id: String,
parseable/src/handlers/http/modal/mod.rs

Line 172 in 6c5f2aa

pub ingestor_id: String,
parseable/src/users/filters.rs

Line 52 in 6c5f2aa

pub id: String,
parseable/src/users/dashboards.rs

Line 38 in 6c5f2aa

pub tile_id: Option<String>,
parseable/src/users/dashboards.rs

Line 103 in 6c5f2aa

pub dashboard_id: Option<String>,
More...
refer https://docs.rs/ulid/1.1.4/ulid/struct.Ulid.html

de-sh added the enhancement label

nitisht added the good first issue label

AkhilKamath commented Jan 23, 2025

I would like to work on this, can I be assigned? @nitisht

de-sh assigned de-sh and AkhilKamath and unassigned de-sh

Contributor Author

de-sh commented Jan 23, 2025

Hi @AkhilKamath, feel free to join our slack if you want to discuss this further

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