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

dont-display-stack-ids #6071

Merged
merged 3 commits into from
Jan 28, 2025
Merged

dont-display-stack-ids #6071

merged 3 commits into from
Jan 28, 2025

Conversation

Copy link

vercel bot commented Jan 24, 2025

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

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 28, 2025 10:50am
gitbutler-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 28, 2025 10:50am

@github-actions github-actions bot added rust Pull requests that update Rust code @gitbutler/desktop labels Jan 24, 2025
@Caleb-T-Owens Caleb-T-Owens force-pushed the dont-display-stack-ids branch from d0ecae6 to 6f85b93 Compare January 24, 2025 00:57
@Caleb-T-Owens Caleb-T-Owens force-pushed the dont-display-stack-ids branch from 6f85b93 to 1dc0c65 Compare January 24, 2025 01:01
@Caleb-T-Owens Caleb-T-Owens force-pushed the dont-display-stack-ids branch from 1dc0c65 to e132eb7 Compare January 24, 2025 01:44
@Caleb-T-Owens Caleb-T-Owens force-pushed the dont-display-stack-ids branch from e132eb7 to 80d7700 Compare January 24, 2025 09:56
@Caleb-T-Owens Caleb-T-Owens force-pushed the dont-display-stack-ids branch from 80d7700 to 30ee14f Compare January 27, 2025 15:28
I ummed and ahhed about this one, because you _could_ use the other API to get the same data, but this API has the built in filtering which the frontend makes use of mre efficiently, and where I need the data in the next commit, it makes much more sense to use this API that I'm modifying here.
I'm looking up the details from the sidebar entry (which should already be loaded) and using the virtualBranchReference if it's present, before falling back to the PR title or remote branch given name.

I also made use of the Badge component for the local badge which lets us tidy up some CSS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@gitbutler/desktop rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Branch search: failing to find branch by exact name
1 participant