Skip to content
This repository has been archived by the owner on Oct 4, 2023. It is now read-only.

[C-1452] Implement release-date field #2246

Merged
merged 4 commits into from
Nov 11, 2022
Merged

Conversation

dylanjeffers
Copy link
Contributor

@dylanjeffers dylanjeffers commented Nov 10, 2022

Description

Implements release-date field

  • Adds react-native-modal-datetime-picker
RPReplay_Final1668107991.mov

@dylanjeffers dylanjeffers requested review from a team and nicoback2 and removed request for a team November 10, 2022 19:23
Comment on lines 20 to 24
return (
date.getDate() === today.getDate() &&
date.getMonth() === today.getMonth() &&
date.getFullYear() === today.getFullYear()
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😅 it really feels like there should be an easier way, but this totally works. No complaints

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There def is I'll use moment

textTransform: 'uppercase'
},
divider: {
marginHorizontal: spacing(-4)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀


return (
<>
<TouchableOpacity style={styles.root} onPress={handlePress}>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: root feels slightly misleading

maybe best fixed by moving the divider up a level?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes good call

@dylanjeffers dylanjeffers merged commit 9e84248 into main Nov 11, 2022
@dylanjeffers dylanjeffers deleted the dj-c-1452-release-date-field branch November 11, 2022 05:32
audius-infra pushed a commit that referenced this pull request Nov 12, 2022
[ad230d1] v1.4.0 (#2242) Sebastian Klingler
[dc11096] Stop propagation on repost button click in tracks tables (#2249) Kyle Shanks
[5964998] Update push notification types, update notif navigation hook, and remove userInteraction check in onNotification method (#2247) Kyle Shanks
[decabd2] [C-1461] Implement license-type/isrc upload flow (#2248) Dylan Jeffers
[9e84248] [C-1452] Implement release-date field (#2246) Dylan Jeffers
[8a0c7a8] Fix race condition checking stream_mp3 feature flag (#2244) Andrew Mendelsohn
[36d53b0] Don't delete sourcemaps on sync to s3 (#2243) Sebastian Klingler
@AudiusProject AudiusProject deleted a comment from linear bot Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants