Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bring dev into main #5

Merged
merged 30 commits into from
Jul 26, 2021
Merged

bring dev into main #5

merged 30 commits into from
Jul 26, 2021

Conversation

steveoh
Copy link
Member

@steveoh steveoh commented Jul 21, 2021 •
edited
Loading

fix ci and a few bugs

merge once we know it's all happy

steveoh added 29 commits July 6, 2021 18:15
@steveoh
fix: update trigger settings
b0d4bb8
@steveoh
fix: add a token to authenticate
104f6ef
@steveoh
fix: set working directory
29667e9
@steveoh
fix: update working directory
b8b77c9
@steveoh
fix: last guess at where the project is
744e776
@steveoh
fix: allow terraform to access GCP
2c09169
@steveoh
fix: try different env variable
e538e27
@steveoh
fix: select workspace
03b736b
@steveoh
refactor: move env to job
1d59bfe
@steveoh
chore: the function needs it's own package.json
340ae7d
@steveoh
fix: only use dotenv in development
4f25efe
@steveoh
fix: update schema to accept current payload
fe01e2c
@steveoh
chore: add querystring
0fd857e
@steveoh
test: add tests
9c03602
@steveoh
chore: spelling
c4a4ee1
@steveoh
ci: add tests to ci
f49d647
@steveoh
ci: fix working directory path
005f493
@steveoh
fix: remove unused dependency
cb71d77
@steveoh
fix: only require dotenv during development
86437f4
@steveoh
fix: order
a2d9fc5
@steveoh
refactor: change exports
bd5d10b
@steveoh
chore: loosen base map name restriction
98ed64f
@steveoh
fix: redline was double encoded
18bb6e8
@steveoh
fix(api): don't double stringify redline
e0399bc
@steveoh
test: update link format
441cf45
@steveoh
chore: lowercase basemap and set a default value
6502cde
@steveoh
Revert "fix(api): don't double stringify redline"
dd73b5b 
This reverts commit e0399bc.
@steveoh
chore: remove terraform tests
cdd5442
@steveoh
ci: remove needs for now
20a4a78
stdavis
stdavis approved these changes Jul 22, 2021
View reviewed changes
Copy link
Member

@stdavis stdavis left a comment

Choose a reason for hiding this comment

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

Seems to be working to me and the code looks good.

@steveoh steveoh temporarily deployed to dev July 26, 2021 15:13 Inactive
@steveoh steveoh merged commit e2cb6c6 into main Jul 26, 2021
@steveoh steveoh deleted the dev branch July 26, 2021 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stdavis stdavis stdavis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@steveoh @stdavis