Merge pull request #7 from agritheory/table-fixes

fixes: consistent row height, payment entry link
agritheory · Aug 7, 2022 · 4911093 · 4911093
2 parents 62363ce + 3c1eaf3
commit 4911093
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/check_run/public/js/check_run/CheckRun.vue b/check_run/public/js/check_run/CheckRun.vue
@@ -86,8 +86,8 @@
 								:id="item.id" />Pay
 						</td>
 						<td v-else>
-							<a :href="paymentEntryUrl(item)" target="_blank">
-							{{ item.check_number }}</a></td>
+							<a target="_blank" :href="paymentEntryUrl(item)">
+							{{ item.payment_entry }}</a></td>
 				</tr>
 				</template>
 			</tbody>
@@ -129,7 +129,7 @@ export default {
 			return encodeURI(`${frappe.urllib.get_base_url()}/app/${transaction.doctype.toLowerCase().replace(" ", "-")}/${transaction.name}`)
 		},
 		paymentEntryUrl: transaction => {
-			if(transaction.payment_entry) { return "" }
+			if(!transaction.payment_entry) { return "" }
 			return encodeURI(`${frappe.urllib.get_base_url()}/app/payment-entry/${transaction.payment_entry}`)
 		},
 		sortTransactions(key) {
@@ -193,4 +193,7 @@ export default {
 	.table tr.selectedRow {
 		background-color: var(--yellow-highlight-color);
 	}
+	.table tr {
+		height: 50px;
+	}
 </style>