search

middlewarehq / middleware

✨ Open-source dev productivity platform for engineering teams ✨
https://middlewarehq.com
Apache License 2.0
390 stars 32 forks source link

Fix 366 #367

Closed gajrajgchouhan closed 1 month ago

gajrajgchouhan commented 1 month ago

Linked Issue(s)

Closes #366

Acceptance Criteria fulfillment

Proposed changes (including videos or screenshots)

Further comments

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.

gajrajgchouhan commented 1 month ago

There is a PullRequestsTableMini as well, do I need to change that as well?

samad-yar-khan commented 1 month ago

@e-for-eshaan Can you please look into these front-end changes ?

e-for-eshaan commented 1 month ago

Hey @gajrajgchouhan, thanks for this PR. The DeploymentInsightsOverlay (the one that opens when you click on the Deployment Frequency card) utilises PullRequestsTableMini. You should remove the redundant copy from there as well.

gajrajgchouhan commented 1 month ago

Hi @e-for-eshaan , I have done this.