-
Notifications
You must be signed in to change notification settings - Fork 0
Separate show queries towards Arcsi #325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from 10 commits
27e3a09
5a1f6f3
305c6d3
6532104
f93e3da
6a10a56
d66ebad
651f202
8dc7bcb
d1b07d1
096fd24
d3788a8
6b04333
94d075f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -53,9 +53,14 @@ export default { | |
|
|
||
| const minutes = today.getMinutes() | ||
|
|
||
| // refresh arcsiShows every 10 minutes | ||
| // refresh arcsiShowsForTiles every 10 minutes | ||
| if (minutes % 10 === 0) { | ||
| this.refreshAllShows() | ||
| this.refreshArcsiShowsForTiles() | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Question about tiles: Is it only going to change when we add new shows or also whenever a new episode is published? Anyway I think schedule 10 min is okay but hopefully we can be more relaxed about shows tiles after this split, what do you think? |
||
| } | ||
|
|
||
| // refresh arcsiShowsForSchedule every 10 minutes | ||
| if (minutes % 10 === 0) { | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. a két |
||
| this.refreshArcsiShowsForSchedule() | ||
| } | ||
|
|
||
| // refresh rareShows every 3 minutes | ||
|
|
@@ -68,14 +73,24 @@ export default { | |
| this.refreshCustomSchedule() | ||
| } | ||
| }, | ||
| async refreshAllShows () { | ||
| await this.$axios.get(arcsiShowsBaseURL + '/all_without_items', config) | ||
| async refreshArcsiShowsForTiles () { | ||
| await this.$axios.get(`${arcsiShowsBaseURL}/all_tiles`, config) | ||
| .then((res) => { | ||
| this.$store.commit('refreshArcsiShowsForTiles', res.data) | ||
| }) | ||
| .catch((e) => { | ||
| this.$sentry.captureException(e) | ||
| this.error({ statusCode: 404, message: 'Arcsi Shows Tiles endpoint not found' }) | ||
| }) | ||
| }, | ||
| async refreshArcsiShowsForSchedule () { | ||
| await this.$axios.get(`${arcsiShowsBaseURL}/all_schedule`, config) | ||
| .then((res) => { | ||
| this.$store.commit('refreshAllShowsList', res.data) | ||
| this.$store.commit('refreshArcsiShowsForSchedule', res.data) | ||
| }) | ||
| .catch((e) => { | ||
| this.$sentry.captureException(e) | ||
| this.error({ statusCode: 404, message: 'All shows endpoint not found' }) | ||
| this.error({ statusCode: 404, message: 'Arcsi Shows Schedule endpoint not found' }) | ||
| }) | ||
| }, | ||
| async refreshRareShows () { | ||
|
|
@@ -85,7 +100,7 @@ export default { | |
| }) | ||
| .catch((e) => { | ||
| this.$sentry.captureException(e) | ||
| this.error({ statusCode: 404, message: 'Rare Shows not found' }) | ||
| this.error({ statusCode: 404, message: 'CMS Rare Shows not found' }) | ||
| }) | ||
| }, | ||
| async refreshCustomSchedule () { | ||
|
|
@@ -95,7 +110,7 @@ export default { | |
| }) | ||
| .catch((e) => { | ||
| this.$sentry.captureException(e) | ||
| this.error({ statusCode: 404, message: 'Custom schedule not found' }) | ||
| this.error({ statusCode: 404, message: 'CMS Custom schedule not found' }) | ||
| }) | ||
| } | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
valóban, itt el lett felejtve frissíteni a readme-t az .nvmrc aktuális állapotáról, köszi! viszont akkor ebben a sorban is frissíteni kellene