search

MightyOrm / Mighty

A new, small, dynamic micro-ORM. Highly compatible with Massive, but with many essential new features.
BSD 3-Clause "New" or "Revised" License
101 stars 20 forks source link

Paged results #21

Open garrjo opened 4 years ago

garrjo commented 4 years ago

I'm attempting to write a Paged Query and I have configured it to do so; however, the yielded result set is atypical of what normally would occur in a result set coming from MighyOrm. Do we have more documentation to give me an example of how exactly is the best practice to work wiith a PagedResult? var tbl = new Mighty.StartingAccounts(); var page = tbl.Paged(currentPage: currentPage, pageSize: 100, columns:"Id"); As I'd like to work with them in a list I'm unfamiliar with how to bring them to it? Since the result is PagedResults ... Can you give me a hint here?

mikebeaton commented 4 years ago

I was hoping this should be fairly self explanatory based on IntelliSense?

The PagedResults object has fields Items, TotalPages and TotalRecords. If you just type . next to a PagedResults object in Visual Studio or VS Code then it should show them to you as auto-complete options? And the fields themselves have IntelliSense.

Does that help? Is IntelliSense all working for you? It should be since v. 3.0.5 and 66f0106 .

garrjo commented 4 years ago

It should but, traction seems to vary. I will take a look at next run

Get Outlook for Androidhttps://aka.ms/ghei36

From: Mike Beaton notifications@github.com Sent: Saturday, March 21, 2020 2:05:44 PM To: MightyOrm/Mighty Mighty@noreply.github.com Cc: Joe Garrett joegarrett@outlook.com; Author author@noreply.github.com Subject: Re: [MightyOrm/Mighty] Paged results (#21)

I was hoping this should be fairly self explanatory based on IntelliSense?

The PagedResults object has fields Items, TotalPages and TotalRecords. If you just type . next to a PagedResults object in Visual Studio or VS Code then it should show them to you as auto-complete options? And the fields themselves have IntelliSense.

Does that help? Is IntelliSense all working for you? It should be since v. 3.0.5https://github.com/MightyOrm/Mighty/tree/3.0.5 and 66f0106https://github.com/MightyOrm/Mighty/commit/66f010624fa2606d0cc43f5a748138dd583bced3 .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/MightyOrm/Mighty/issues/21#issuecomment-602088481, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AADZAFZVTDVD77N73EMIYLLRIUFYRANCNFSM4LQ6KNHA.

mikebeaton commented 4 years ago

@garrjo I'm gathering that what would have helped here is just a bit more detail in the current docs on paging, right?

garrjo commented 4 years ago

Absolutely,. Having an example to go by beyond the implementation. A Use case would definitely given direction as to how exactly to implement

Get Outlook for Androidhttps://aka.ms/ghei36

From: Mike Beaton notifications@github.com Sent: Sunday, March 22, 2020 1:18:43 AM To: MightyOrm/Mighty Mighty@noreply.github.com Cc: Joe Garrett joegarrett@outlook.com; Mention mention@noreply.github.com Subject: Re: [MightyOrm/Mighty] Paged results (#21)

@garrjohttps://github.com/garrjo I'm gathering that what would have helped here is a bit more detail in the current docs on paginghttps://mightyorm.github.io/Mighty/docs/paging.html, right?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/MightyOrm/Mighty/issues/21#issuecomment-602153820, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AADZAF4IZ3ZWE5UIRGWQOPDRIWUUHANCNFSM4LQ6KNHA.

mikebeaton commented 4 years ago

Fair point! :)

I'm trying to match - but not necessarily exceed, given available time and resources! - the detail of the docs for Massive. So I agree I should document the fields that are in PagedResults (in the docs, not just in IntelliSense!), and also the reason for having them; namely, that you can't show a user paging info unless the call to the paging method returns the total number of pages/items along with the items for the current page.

So I'll leave this Issue open pending adding this info to the docs.

garrjo commented 4 years ago

Sounds great. Thank you for your work and time.

Get Outlook for Androidhttps://aka.ms/ghei36

From: Mike Beaton notifications@github.com Sent: Monday, March 23, 2020 4:21:52 AM To: MightyOrm/Mighty Mighty@noreply.github.com Cc: Joe Garrett joegarrett@outlook.com; Mention mention@noreply.github.com Subject: Re: [MightyOrm/Mighty] Paged results (#21)

Fair point! :)

I'm trying to match - but not necessarily exceed, given available time and resources! - the detail of the docs for Massive. So I agree I should document the fields that are in PagedResults, and also the reason for having them (namely, that you can't show a user paging info unless the call to the paging method returns the total number of pages/items along with the items for the current page).

So I'll leave this Issue open pending adding that info to the docs.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/MightyOrm/Mighty/issues/21#issuecomment-602476627, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AADZAFYHIEQB2YEDMJEVEUDRI4S3BANCNFSM4LQ6KNHA.