Bring back page edit history

[vnugent]

Use case

When viewing a climb/area page I want to see the edit history

This feature was already implemented and available in the old area page. We just need to port it over.

Old area page: https://openbeta.io/areas/18c5dd5c-8186-50b6-8a60-ae2948c548d1 (click on Edit -> Histor)
New ara page: https://openbeta.io/crag/18c5dd5c-8186-50b6-8a60-ae2948c548d1

[actuallyyun]

Hey I don't think the first link is working :

[vnugent]

Thanks for catching this. The test area only exists in production. In development your local app connects to the staging api server which doesn't have the same area id. Maybe we can make the area link configurable, 1 for development and 1 for production.

[saferthanhouses]

Happy to look into this. Are there any designs available?

[saferthanhouses]

Or possibly a simple

And showing the history popup from the old page on click

[vnugent]

@saferthanhouses thank you for taking on this.

Pointers:

• Old area edit modal trigger:

  open-tacos/src/components/edit/AreaEditTrigger.tsx

  Lines 86 to 90 in c22c0a2

  	<MobileDialog modal open={action === 'history'} onOpenChange={postCreateUpdateHandler}>
  	<DialogContent title='Change history'>
  	<RecentChangeHistory history={history} />
  	</DialogContent>
  	</MobileDialog>

• Edit mode toggle: https://github.com/OpenBeta/open-tacos/blob/develop/src/components/crag/cragSummary.tsx#L301-L303
• You can just add a small button next to the edit toggle for now. The toggle is the primary call-to-action so please make the new button less prominent
• we can ask for some UX help and refine this page later in a separate PR

email me viet at openbeta dot io for additional API keys to work with auth0

[saferthanhouses]

@vnugent I'm working on updating this PR so the ChangeHistory data is pulled dynamically - not from getStaticProps as it is currently.

[vnugent]

@saferthanhouses thanks for picking up this task.
Do you think there's a huge benefit in pulling the data dynamically since the majority of our users simple read the pages?

[saferthanhouses]

@vnugent possibly not! But it was a good opportunity to dig into the different SSR strategies the app is using 😆
PR reopened.