illuminodes / minions

A framework for building performant and secure PWAs.
BSD 3-Clause "New" or "Revised" License
6 stars 5 forks source link

Add AG Grid WASM Bindings and Component Coverage #2 #6

Closed rahat2134 closed 4 weeks ago

rahat2134 commented 1 month ago

Implemented AG Grid WASM bindings and Yew component integration with comprehensive coverage for simple table structures. (Fix #2 - AG grid)

Features Added

Implementation Details

  1. AG Grid Bindings (ag_grid.rs):

    • Core grid creation and manipulation
    • Column definitions with customization options
    • Default column properties
    • Grid options configuration
    • Type-safe option builders
  2. Yew Component (component.rs):

    • Reusable functional component
    • Props for data, columns, and configuration
    • Auto grid initialization and updates
    • Responsive layout support

Files Changed

Features

Closes #2

rahat2134 commented 1 month ago

@42Pupusas Please take a look!

rahat2134 commented 1 month ago

@42Pupusas PTAL to this after this I will tackle others on the list... For e.g., leaflets and calendar

42Pupusas commented 1 month ago

Hey @rahat2134 this bindings and helpers are looking great.

Please note the bounty description in #2 includes:

Demo of Yew functional components showcasing Nostr integration

A simple test component that puts kind 0 or 1 events in a table will suffice.

You can see an example of pulling notes from the relay pool in the code I committed today.

rahat2134 commented 1 month ago

@42Pupusas PTAL! I have pushed the changes.

42Pupusas commented 1 month ago

Im getting a browser panic when the table is created, can we make sure all the bindings are implemented correctly?


ag-grid-community.min.js:1 Uncaught TypeError: e.map is not a function
    at u.setRowData (ag-grid-community.min.js:1:1713)
    at p.setRowData (ag-grid-community.min.js:1:20629)
    at y.onRowDataUpdated (ag-grid-community.min.js:1:28291)
    at ag-grid-community.min.js:1:27064
    at ag-grid-community.min.js:1:43663
    at kg.wrapIncoming (ag-grid-community.min.js:1:669896)
    at ag-grid-community.min.js:1:43645
    at ag-grid-community.min.js:1:43703
    at Set.forEach (<anonymous>)
    at r (ag-grid-community.min.js:1:43552)

```
rahat2134 commented 1 month ago

@42Pupusas PTAL! Any more suggestions?

rahat2134 commented 4 weeks ago

@42Pupusas , Pushed the changes. PTAL!

Screenshot 2024-10-28 at 2 56 58 AM
42Pupusas commented 4 weeks ago

Everything looks and works great. Another great contribution @rahat2134 .

rahat2134 commented 3 weeks ago

@42Pupusas, Is the transaction for this bounty completed?

rahat2134 commented 3 weeks ago

@42Pupusas PTAL whenever you have some bandwidth.