Dynamic removal of blocks

[DivadNojnarg]

Will fix one of the issue in #136, where the remove button is missing for dynamically added blocks.

[codecov]

Codecov Report

Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (cbb1b67) 60.73% compared to head (aa7f166) 80.75%.

Files	Patch %	Lines
R/server.R	27.27%	16 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #139       +/-   ##
===========================================
+ Coverage   60.73%   80.75%   +20.02%     
===========================================
  Files          16       16               
  Lines        1663     1663               
===========================================
+ Hits         1010     1343      +333     
+ Misses        653      320      -333     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[DivadNojnarg]

@nbenn : unit tests will follow in another PR. I already have some of them in #119 and don't want to copy and paste here.

[DivadNojnarg]

Note @nbenn: I can also make handle_remove_block an internal function instead of S3, if we assume people won't need to customise it, which is likely.

[nbenn]

Note @nbenn: I can also make handle_remove_block an internal function instead of S3, if we assume people won't need to customise it, which is likely.

@DivadNojnarg You could also not export the generic function itself I believe. The corresponding (class-specific) methods have to be exported, but as long as the generic is not, it should behave as "interal" function, despite using s3 dispatch.

[nbenn]

No objections from my side.

[DivadNojnarg]

Great @nbenn. I am done with this PR. This can go to main.

[nbenn]

Lgtm.