Browse Source

add some other image deletions in other cases

photo-upload
Trent Larson 6 months ago
parent
commit
c602c5ce50
  1. 4
      project.task.yaml
  2. 18
      src/views/GiftedDetails.vue
  3. 11
      src/views/GiftedPhoto.vue

4
project.task.yaml

@ -18,6 +18,10 @@ tasks :
- upload to a public readable place at correct hosting location - upload to a public readable place at correct hosting location
- remove previous image when editing - remove previous image when editing
- on gift details, if project then show, otherwise mark "gift for you" - on gift details, if project then show, otherwise mark "gift for you"
- ping endpoint
- limits endpoint
- 08 image on profile
- ask to detect location & record it in settings - ask to detect location & record it in settings
- if personal location is set, show potential local affiliations - if personal location is set, show potential local affiliations

18
src/views/GiftedDetails.vue

@ -8,7 +8,7 @@
<div class="text-lg text-center font-light relative px-7"> <div class="text-lg text-center font-light relative px-7">
<h1 <h1
class="text-lg text-center px-2 py-1 absolute -left-2 -top-1" class="text-lg text-center px-2 py-1 absolute -left-2 -top-1"
@click="$router.back()" @click="cancel()"
> >
<fa icon="chevron-left" class="fa-fw"></fa> <fa icon="chevron-left" class="fa-fw"></fa>
</h1> </h1>
@ -184,6 +184,7 @@ export default class GiftedDetails extends Vue {
} }
cancel() { cancel() {
this.deleteImage(); // not awaiting, so they'll go back immediately
this.$router.back(); this.$router.back();
} }
@ -212,15 +213,8 @@ export default class GiftedDetails extends Vue {
}, },
); );
if (response.status === 204) { if (response.status === 204) {
this.$notify( // don't bother with a notification
{ // (either they'll simply continue or they're canceling and going back)
group: "alert",
type: "success",
title: "Deleted",
text: "That image record was deleted.",
},
5000,
);
} else { } else {
console.error("Error deleting image:", response); console.error("Error deleting image:", response);
this.$notify( this.$notify(
@ -232,6 +226,7 @@ export default class GiftedDetails extends Vue {
}, },
5000, 5000,
); );
// keep the imageUrl in localStorage so the user can try again if they want
return; return;
} }
localStorage.removeItem("imageUrl"); localStorage.removeItem("imageUrl");
@ -328,8 +323,9 @@ export default class GiftedDetails extends Vue {
title: "Success", title: "Success",
text: `That ${this.isTrade ? "trade" : "gift"} was recorded.`, text: `That ${this.isTrade ? "trade" : "gift"} was recorded.`,
}, },
7000, 5000,
); );
localStorage.removeItem("imageUrl");
this.$router.back(); this.$router.back();
} }
// eslint-disable-next-line @typescript-eslint/no-explicit-any // eslint-disable-next-line @typescript-eslint/no-explicit-any

11
src/views/GiftedPhoto.vue

@ -154,15 +154,8 @@ export default class GiftedPhoto extends Vue {
); );
this.uploading = false; this.uploading = false;
this.$notify( // we won't give a notification here because the user will be taken back to finish
{
group: "alert",
type: "success",
title: "Stored",
text: "Your picture has been stored.",
},
3000,
);
localStorage.setItem("imageUrl", response.data.url as string); localStorage.setItem("imageUrl", response.data.url as string);
this.$router.back(); this.$router.back();
} catch (error) { } catch (error) {

Loading…
Cancel
Save