Add promo video to form #1292
Add promo video to form #1292
Conversation
| @@ -701,11 +702,20 @@ class Meta: | |||
|
|
|||
| class RecruitmentSerializer(CustomBaseSerializer): | |||
| seperate_positions = RecruitmentSeperatePositionSerializer(many=True, read_only=True) | |||
| promo_media = serializers.CharField(max_length=100) | |||
There was a problem hiding this comment.
Tror ikke dette er nødvendig, ettersom det er allerede gjort gjennom at det er et innkludert felt
There was a problem hiding this comment.
Det var nødvendig for å tillate input lenger enn 11 characters i api-interface
There was a problem hiding this comment.
Kan gjøres i 1 migrasjonsfil :D
| max_applications: 'max_applications', | ||
| promo_media: 'promo_video', |
There was a problem hiding this comment.
feil navngivning, se toppen av filen
| [KEY.max_applications]: 'Max applications per user', | ||
| [KEY.promo_media]: "Promo Video", |
| def validate_promo_media(self, value: str | None) -> str | None: | ||
| if (value is None): | ||
| return None | ||
| match = re.search(r'(youtu.*be.*)\/(watch\?v=|embed\/|v|shorts|)(.*?((?=[&#?])|$))', value) | ||
| if (match): | ||
| return match.group(3) | ||
| raise ValidationError("Invalid youtube url") | ||
|
|
There was a problem hiding this comment.
Hvor brukes denne? Bør nok være i recruitment og ikke serializer. Validate blir brukt i andre steder i recruitment model objektet, kan sette dette opp der
There was a problem hiding this comment.
Kan i samme sleng påkreve kun siste biten av url, og så manuellt legge den til når den vises
There was a problem hiding this comment.
Den kjøres av seg selv, django magic ellerno
There was a problem hiding this comment.
…eate-frontend-for-submitting-and-displaying-recruitment-promo-media
| }; | ||
|
|
||
| function youtubeLinkValidator(state: FormType): SamfError { |
There was a problem hiding this comment.
Anbefaler å holde komponentfilen ren. Kan flyttes til ./utils.ts
No description provided.