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

Feat/um 49 query builder api #1573

Merged
merged 6 commits into from
Oct 22, 2022
Merged

Feat/um 49 query builder api #1573

merged 6 commits into from
Oct 22, 2022

Conversation

briancao
Copy link
Collaborator

query event data

@vercel
Copy link

vercel bot commented Oct 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
analytics ✅ Ready (Inspect) Visit Preview Oct 22, 2022 at 2:46AM (UTC)
umami ✅ Ready (Inspect) Visit Preview Oct 22, 2022 at 2:46AM (UTC)

@briancao briancao requested a review from mikecao October 21, 2022 23:56
@netlify
Copy link

netlify bot commented Oct 21, 2022

Deploy Preview for heartfelt-souffle-72e1df failed.

Name Link
🔨 Latest commit ec42ad5
🔍 Latest deploy log https://app.netlify.com/sites/heartfelt-souffle-72e1df/deploys/6353587d86b82c000961a1b6

Copy link
Collaborator

@mikecao mikecao left a comment

Choose a reason for hiding this comment

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

Looks good, just a few minor things.

};

const handleRemoveTag = (value, list, setState) => {
const { ...rest } = list;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Makes no sense to spread the list, just use the list.

setData(data);

if (message) {
setMessage(null);
Copy link
Collaborator

Choose a reason for hiding this comment

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

No need to check, just always set it null on success.

Copy link
Contributor

@essesoul essesoul left a comment

Choose a reason for hiding this comment

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

wow that's awesome

@briancao briancao merged commit ba31f48 into dev Oct 22, 2022
@briancao briancao deleted the feat/um-49-query-builder-api branch October 22, 2022 04:33
dannyhines pushed a commit to dannyhines/umami that referenced this pull request Nov 9, 2022
* add uuid to event. add indexes

* eventdata api

* add event data

* remove test data

* update list
benithors pushed a commit to benithors/umami that referenced this pull request Nov 10, 2022
* add uuid to event. add indexes

* eventdata api

* add event data

* remove test data

* update list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants